Closed Bug 1022573 Opened 6 years ago Closed 3 years ago

Australis widget icons lack contrast when using High Contrast mode

Categories

(Firefox :: Theme, defect, P4)

30 Branch
x86_64
Windows 8.1
defect

Tracking

()

VERIFIED FIXED
Firefox 51
Tracking Status
firefox51 --- verified

People

(Reporter: Unfocused, Assigned: dao)

References

(Blocks 1 open bug)

Details

(Whiteboard: p=3)

Attachments

(2 files, 1 obsolete file)

The widget icons added for Australis lack adequate contrast when using High Contrast mode. This is especially true of the cut/copy/paste and zoom controls. See attached screenshot. We should have black versions and white versions, for black-on-white and white-on-black high Contrast modes respectively.
Flags: firefox-backlog+
Attached image Screenshot
(In reply to Blair McBride [:Unfocused] from comment #0)
> The widget icons added for Australis lack adequate contrast when using High
> Contrast mode. This is especially true of the cut/copy/paste and zoom
> controls. See attached screenshot.

The zoom controls have the same contrast as other items in your screenshot. Cut/copy/paste are probably just disabled.
Priority: -- → P4
Attached patch patch (obsolete) — Splinter Review
Assignee: nobody → dao+bmo
Status: NEW → ASSIGNED
Attached patch patch v2Splinter Review
Attachment #8778522 - Attachment is obsolete: true
Attachment #8778531 - Flags: review?(mconley)
This looks okay, but I've pushed a mozscreenshot job to closer inspection:

https://treeherder.mozilla.org/#/jobs?repo=try&revision=f47847a40c78
Comment on attachment 8778531 [details] [diff] [review]
patch v2

Review of attachment 8778531 [details] [diff] [review]:
-----------------------------------------------------------------

Sorry for the delay - the screenshots stuff didn't end up working out on automation, so I did a bunch of manual testing locally. This looks good to me! Thanks for your work on this stuff, Dao.

::: browser/themes/shared/menupanel.inc.css
@@ -24,5 @@
>    toolbarpaletteitem[place="palette"] > #bookmarks-menu-button {
>      -moz-image-region: rect(0px, 192px, 32px, 160px);
>    }
>  
> -  #bookmarks-menu-button[cui-areatype="menu-panel"][panel-multiview-anchor=true] {

It's nice that we can get rid of these and do CSS filtering instead. Perhaps we should file a bug to remove the unnecessary glyphs from menuPanel.png and menuPanel@2x.png as well?
Attachment #8778531 - Flags: review?(mconley) → review+
(In reply to Mike Conley (:mconley) - (Needinfo me!) from comment #6)
> > -  #bookmarks-menu-button[cui-areatype="menu-panel"][panel-multiview-anchor=true] {
> 
> It's nice that we can get rid of these and do CSS filtering instead. Perhaps
> we should file a bug to remove the unnecessary glyphs from menuPanel.png and
> menuPanel@2x.png as well?

This patch actually does that already.
(In reply to Dão Gottwald [:dao] from comment #7)
> (In reply to Mike Conley (:mconley) - (Needinfo me!) from comment #6)
> > > -  #bookmarks-menu-button[cui-areatype="menu-panel"][panel-multiview-anchor=true] {
> > 
> > It's nice that we can get rid of these and do CSS filtering instead. Perhaps
> > we should file a bug to remove the unnecessary glyphs from menuPanel.png and
> > menuPanel@2x.png as well?
> 
> This patch actually does that already.

My apologies - I hadn't checked the patch, only the diff. Thanks, and carry on!
Pushed by dgottwald@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/6af49d08884d
Make icons in the app menu work better with high-contrast settings. r=mconley
https://hg.mozilla.org/mozilla-central/rev/6af49d08884d
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 51
Depends on: 1295502
I have reproduced this bug with Nightly 32.0a1 (2014-06-09) on Windows 8.1 , 64Bit!

This Bug's fix is verified on Latest Nightly  51.0a1.
Build ID : 20160816030459
User Agent : Mozilla/5.0 (Windows NT 6.3; WOW64; rv:51.0) Gecko/20100101 Firefox/51.0

[bugday-20160817]
Depends on: 1297806
No longer depends on: 1295502
Blocks: 1304341
You need to log in before you can comment on or make changes to this bug.