1732908 - The context menu text is not visible while navigate through the menu with high contrast white on Windows 7

Status: NEW

(Firefox :: Theme, defect)

Description

[Alin Ilea, Desktop Test Engineering]

Attachment: 2021-09-28_16h05_04.mp4

Tested with:

Release 92
Beta 93
Nightly 94.0a1 (2021-09-28)

Tested on:

Windows 7

Preconditions:

Have the High Contrast White enabled

Steps:

1. Launch Firefox and right click anywhere in order to open the context menu.
2. Navigate through the context menu.

Actual result:

The context menu text is not visible at all while navigate through the menu.

Expected result:

The text should be white on black background. The context menu text should change the color to white while hovering over the text, like the buttons from about:preferences. (see the video attached).

Regression range:

Will try to find the regression range (if its a regression) asap.

Note: This is an intermittent issue. I cannot reproduce it every time.

Comment 1

[Mike Conley (:mconley) (:⚙️)]

Hey mtigley, do you think something like this should go into the MSU Capstone backlog?

Comment 2

[Micah [:mtigley] (she/her)]

(In reply to Mike Conley (:mconley) (:⚙️) from comment #1)

Hey mtigley, do you think something like this should go into the MSU Capstone backlog?

This looks like a good one to pick when we can. I'll add it in, thanks!

Comment 3

[Noah Pesta]

I cannot figure out why the color: -moz-menuhovertext is not being set right when high contrast is enabled. :emilio, could you provide some assistance?

Comment 4

[Emilio Cobos Álvarez [:emilio]]

So that color comes from here. I'd look into:

• Is IsAppThemed returning true / are we entering that codepath?
• If it is, is GetColorFromTheme succeeding?

If the answer to any of those is "no", then we probably need to tweak code for HCM. Maybe that IsAppThemed should also look into high-contrast mode? Maybe we need to get the color in some other way?

Let me know if that helps. Also, does this reproduce on Windows 10 out of curiosity?

Comment 5

[Noah Pesta]

(In reply to Emilio Cobos Álvarez (:emilio) from comment #4)

So that color comes from here. I'd look into:

• Is IsAppThemed returning true / are we entering that codepath?
• If it is, is GetColorFromTheme succeeding?

If the answer to any of those is "no", then we probably need to tweak code for HCM. Maybe that IsAppThemed should also look into high-contrast mode? Maybe we need to get the color in some other way?

Let me know if that helps. Also, does this reproduce on Windows 10 out of curiosity?

Thanks for the help! I will take a look at where the color is coming from. I also just checked and this does not reproduce on Windows 10 Home Version 10.0.19043 Build 19043

Comment 6

[BugBot [:suhaib / :marco/ :calixte]]

The bug assignee is inactive on Bugzilla, so the assignee is being reset.