Closed Bug 661288 Opened 10 years ago Closed 10 years ago

Check mark not visible when hovering a disabled checked menuitem

Categories

(Toolkit :: Themes, defect)

All
macOS
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla7

People

(Reporter: florian, Assigned: florian)

References

Details

(Keywords: polish)

Attachments

(1 file)

Attached patch PatchSplinter Review
It seems this problem appeared with http://hg.mozilla.org/mozilla-central/rev/9b4420523d0c / Bug 542192.

The icon used for hovered checked menuitems has the same color as the background of not-hovered items, which makes it completely invisible for disabled items that aren't highlighted on hover.
Attachment #536654 - Flags: review?(dao)
Attachment #536654 - Flags: review?(dao) → review+
Keywords: checkin-needed
Flags: in-testsuite?
Keywords: checkin-needed
Whiteboard: [fixed in cedar]
Status: NEW → ASSIGNED
Pushed:
http://hg.mozilla.org/mozilla-central/rev/a2007c44b669
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Whiteboard: [fixed in cedar]
Target Milestone: --- → mozilla7
How can I verify this? Can you please provide some STR.
Thanks
(In reply to Vlad [QA] from comment #2)
> How can I verify this? Can you please provide some STR.
> Thanks

The only STR I have are with Instantbird:
- Have a contact list with at least 2 different tags.
- Hide a tag.
- In the context menu of a tag, open the "Visible Tags…" sub-menu.
- The last item of that sub-menu ("Other Tags") is both checked and disabled. When hovering it, the check mark disappears.
This can be reproduced with Instantbird 1.0 and the current nightlies (still on Mozilla 6). We haven't produced mozilla7-based nightlies yet, so it's not possible to verify the fix right now with these STR without compiling oneself.

I tried to create a data URL for a XUL document containing a disabled checked menuitem to provide a very easy way to reproduce, but XUL documents are apparently no longer supported in data URLs :(.
Hi Florian.
You said in comment3 that this can't be verify yet.
Is there a fix a some new STR?
Thanks
You need to log in before you can comment on or make changes to this bug.