Main tab names from preferences are not visible on Windows 7, when using a High Contrast theme

VERIFIED FIXED in Firefox 56

Status

()

defect
P1
normal
VERIFIED FIXED
2 years ago
2 years ago

People

(Reporter: miriapodzemos.my, Assigned: evanxd)

Tracking

56 Branch
Firefox 56
Unspecified
Windows 7
Points:
---
Bug Flags:
qe-verify +

Firefox Tracking Flags

(firefox56 verified)

Details

(Whiteboard: [photon-preference])

Attachments

(2 attachments)

(Reporter)

Description

2 years ago
[Affected versions]: 
Nightly 56.0a1, Build ID 20170723100358

[Affected platforms]:
Platforms: Windows 7 x 64 

[Steps to reproduce]:
1. In Windows 7 go to Control Panel\All Control Panel Items\Personalization and select High Contrast Theme
2. Launch Nightly
3. Go to "about:preferences" and check the main tabs content
4. Go to Firefox Account tab

[Expected result]:
Preferences main Tabs(Icon and name) should be properly displayed on Windows 7, when using a High contrast theme (screenshot section 2)
Android and iOS Logos are properly displayed when using a High contrast theme(screenshot section 3)

[Actual result]:
Main Tabs names from preferences are not visible on Windows 7, when using a High Contrast theme
Android and iOS Logos are not visible when using a High contrast theme

Also, please note that the "Privacy & Security" tab icon and name is not visible at all when High Contrast theme is enabled(screenshot section 1)

For more details please check the attached screenshot. Thank you
(Reporter)

Comment 1

2 years ago
I am not sure if this issue is a Photon Preferences bug, but please take a look. Thank you.
Whiteboard: [photon-preference][triage]
Flags: qe-verify+
Priority: -- → P2
Target Milestone: --- → Firefox 56
Assignee: nobody → evan
Status: NEW → ASSIGNED
Priority: P2 → P1
Comment hidden (mozreview-request)
(Assignee)

Updated

2 years ago
Attachment #8890796 - Flags: review?(mconley)
(Assignee)

Comment 4

2 years ago
Hi Mike,

Could you review the patch?

Thank you.

Comment 5

2 years ago
mozreview-review
Comment on attachment 8890796 [details]
Bug 1384554 - Use list-style-image style for images to support high contrast mode.

https://reviewboard.mozilla.org/r/162000/#review167406

Seems okay, but you might want to do [a screenshot comparison](https://developer.mozilla.org/en-US/docs/Mozilla/QA/Browser_screenshots) on try.

::: browser/themes/windows/preferences/in-content-new/preferences.css
(Diff revision 2)
> -  #category-application > .category-icon {
> -    list-style-image: url("chrome://browser/skin/preferences/in-content-new/icons.svg#applications-native");
> -  }
> -
>    #category-privacy > .category-icon {
> -    list-style-image: url("chrome://browser/skin/preferences/in-content-new/icons.svg#privacy-native");
> +    list-style-image: url("chrome://browser/skin/preferences/in-content-new/icons.svg#security-native");
>    }
>  
>    #category-sync > .category-icon {
>      list-style-image: url("chrome://browser/skin/preferences/in-content-new/icons.svg#sync-native");
>    }
>  
> -  #category-advanced > .category-icon {
> -    list-style-image: url("chrome://browser/skin/preferences/in-content-new/icons.svg#advanced-native");
> +  #category-search > .category-icon {
> +    list-style-image: url("chrome://browser/skin/preferences/in-content-new/icons.svg#search-native");
> -  }
> -
> -  #category-search-results > .category-icon {
> -    list-style-image: url("chrome://browser/skin/preferences/in-content-new/icons.svg#searchResults-native");

I guess these are kinda unrelated fix-ups since bug 1377163 landed?
Attachment #8890796 - Flags: review?(mconley) → review+
(Assignee)

Comment 6

2 years ago
Thank you for reviewing, Mike. We'll verify the change.

Hi Ricky,

Could you help verify it since I'm still busy on working other bugs.

Thank you.
Flags: needinfo?(rchien)
I've verified this patch on Windows 10. Those reported high contrast theme issues are fixed. Thanks!
Flags: needinfo?(rchien)

Comment 8

2 years ago
Pushed by rchien@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/18525ab19366
Use list-style-image style for images to support high contrast mode. r=mconley

Comment 9

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/18525ab19366
Status: ASSIGNED → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → FIXED
Duplicate of this bug: 1370065
(Reporter)

Comment 11

2 years ago
Tested on Windows 7 x 64, with FF Nightly 56.0a1(2017-07-30). The issue is not reproducible. I will mark this as Verified Fixed. Thank you
Status: RESOLVED → VERIFIED
Whiteboard: [photon-preference][triage] → [photon-preference]
You need to log in before you can comment on or make changes to this bug.