[macOS] Color tone inconsistency on links in about:protections page when system dark theme is set
Categories
(Firefox :: Protections UI, defect, P3)
Tracking
()
People
(Reporter: asoncutean, Assigned: brent)
Details
(Keywords: good-first-bug)
Attachments
(2 files)
[Affected versions]:
- 70.0.1
- 71.0b10
- 72.0a1 (2019-11-18)
[Affected platforms]:
- macOS 10.14
- macOS 10.15
[Steps to reproduce]:
Preconditions:
- From System Preferences => General => Appearance set OS system to Dark
- Sign in to FxA using a compromised acccount (eg. login-manager-monitor@mailinator.com password: loginmanagermonitor)
- Sign in to https://monitor.firefox.com/ with the same account
- Access "about:protections" in a new tab
- Observe "How it works" and "Manage devices" links
[Expected result]:
- Both links use the same blue color tone
[Actual result]:
- Links have different blue color tone
[Regression range]:
- Not a regression, reproducible with Fx 70 cycle.
[Additional notes]:
- If step 2 is not executed this issue is not reproducible
Comment 1•4 years ago
•
|
||
I believe the dark theme is behaving as expected, and the light theme is not. After a link has been visited it will change color to indicate it's visited state.
This may be a bit of a design issue.
The links are --in-content-link-color, and after visited they are --in-content-link-color-visited
on light theme both of these are blue-60,
on dark theme:
--in-content-link-color-visited = blue-60
--in-content-link-color = blue-40
Since we are using variables that are across all of product in this case, this should affect other pages as well. I'll look into this and ni: epang to double check what our design system defines the correct colors to be.
edit: Confirmed, visiting about:rights and clicking the "Mozilla Public License" link has the same effect in dark theme vs light theme.
Comment 2•4 years ago
|
||
Just checked what we do on New Tab. The link and visited link colours are the same.
Can we check what colours are being used for the linked colour and the pressed state in dark mode?
Should be Blue 40 for link/visited and Blue 50 for link pressed.
Updated•4 years ago
|
Comment 3•4 years ago
|
||
Spoke with Stephen about this. Never change the colour of the link visited or not. Thanks!
Comment 4•4 years ago
|
||
Thank you, so: --in-content-link-color-visited
should be blue-40
on dark theme.
By the way, this is the OS dark theme for mac (since we had some confusion about that earlier in-person).
Updated•4 years ago
|
Assignee | ||
Comment 5•4 years ago
|
||
I'm looking for a good first bug and am pretty confident I can take a stab at this one, if it's still open and unassigned?
Comment 6•4 years ago
|
||
(In reply to Brent Schroeter from comment #5)
I'm looking for a good first bug and am pretty confident I can take a stab at this one, if it's still open and unassigned?
Hi Brent, yes, it is open and unassigned, feel free to work on it and send me any questions you have.
Assignee | ||
Comment 7•4 years ago
|
||
Thanks! Should I tag you as reviewer when I submit to Phabricator?
Comment 8•4 years ago
|
||
(In reply to Brent Schroeter from comment #7)
Thanks! Should I tag you as reviewer when I submit to Phabricator?
Yep, I'm happy to review it.
Assignee | ||
Comment 9•4 years ago
|
||
The light theme sets both visited and non-visited links to the same color
(--blue-60
) by default, while the dark theme sets non-visited links to
--blue-40
but did not specify a color for visited links, displaying them as
--blue-60
. This resulted in inconsistent styling of pages written in XHTML
versus HTML, e.g. about:protections
versus about:newtab
. epang confirmed
that the desired behavior is to display links as --blue-40
, visited or not.
Updated•4 years ago
|
Comment 10•4 years ago
|
||
Pushed by ntim.bugs@gmail.com: https://hg.mozilla.org/integration/autoland/rev/3bc0d683a41c Correctly set a:visited color in dark-mode. r=ewright
Comment 11•4 years ago
|
||
bugherder |
Comment 12•4 years ago
|
||
Since the status are different for nightly and release, what's the status for beta?
For more information, please visit auto_nag documentation.
Updated•4 years ago
|
Updated•4 years ago
|
Reporter | ||
Comment 13•4 years ago
|
||
The issue is verified fixed with Fx 74.0b2 and Fx 75.0a1(2020-02-12) on macOS 10.14 and macOS 10.15.
Comment 14•4 years ago
|
||
Does the "never change the color of the link visited or not" rule mentioned in Comment 3 applies also to the High Contrast mode on Windows?
Comment 15•4 years ago
|
||
(In reply to Simona Badau from comment #14)
Does the "never change the color of the link visited or not" rule mentioned in Comment 3 applies also to the High Contrast mode on Windows?
No, not for High Contrast mode on Windows. Thanks for the clarifying questions!
Description
•