### Beta Uplift Approval Request __User impact if declined:__ Windows users will be affected since the menubar items don't have the correct colour in inactive windows __Is this code covered by automated tests?:__ No __Has the fix been verified in Nightly?:__ No __Needs manual test from QE?:__ No __List of other uplifts needed:__ None __Risk to taking this patch:__ Low __Why is the change risky/not risky? (and alternatives if risky):__ This patch adds a style back in that was previously removed due to a misunderstanding (only the XP_WIN condition wrapped around it should have been removed) __String changes made/needed: __ No ### Beta/Release Uplift Approval Request * **User impact if declined**: Windows users will be affected since the menubar items don't have the correct color in inactive windows. * **Is this code covered by automated tests?**: No * **Has the fix been verified in Nightly?**: No * **Needs manual test from QE?**: No * **If yes, steps to reproduce**: * **List of other uplifts needed**: None * **Risk to taking this patch**: Low * **Why is the change risky/not risky? (and alternatives if risky)**: Its adding one style back that was removed by accident (the condition wrapping it was supposed to be removed, not the entire style) but we didn't catch until FX100 moved to Beta. * **String changes made/needed**: No
Bug 1762913 Comment 9 Edit History
Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.
### Beta/Release Uplift Approval Request * **User impact if declined**: Windows users will be affected since the menubar items don't have the correct color in inactive windows. * **Is this code covered by automated tests?**: No * **Has the fix been verified in Nightly?**: No * **Needs manual test from QE?**: No * **If yes, steps to reproduce**: * **List of other uplifts needed**: None * **Risk to taking this patch**: Low * **Why is the change risky/not risky? (and alternatives if risky)**: Its adding one style back that was removed by accident (the condition wrapping it was supposed to be removed, not the entire style) but we didn't catch until FX100 moved to Beta. * **String changes made/needed**: No
### Beta/Release Uplift Approval Request * **User impact if declined**: Windows users will be affected since the menubar items don't have the correct color in inactive windows. * **Is this code covered by automated tests?**: No * **Has the fix been verified in Nightly?**: No * **Needs manual test from QE?**: Yes * **If yes, steps to reproduce**: In Windows, check that the menubar items have the expected/correct colour in inactive windows * **List of other uplifts needed**: None * **Risk to taking this patch**: Low * **Why is the change risky/not risky? (and alternatives if risky)**: Its adding one style back that was removed by accident (the condition wrapping it was supposed to be removed, not the entire style) but we didn't catch until FX100 moved to Beta. * **String changes made/needed**: No