Closed Bug 1417927 Opened 7 years ago Closed 3 years ago

Current active tab is hard to catch with photon dark theme

Categories

(Firefox :: Theme, defect, P3)

defect

Tracking

()

RESOLVED INVALID

People

(Reporter: gerard-majax, Assigned: shorlander)

References

(Blocks 1 open bug)

Details

(Whiteboard: [photon-followup])

Attachments

(2 files)

Attached image dark theme
It has been reported to me that the current active tab's contract is low enough that it is hard to distinguish it when using the Photon Dark Theme. At least on Windows 8.1.
Attached image light theme
Stephen, should we tweak these colors?
Flags: needinfo?(shorlander)
Priority: -- → P3
No, there is a full color stop between the background and foreground color. There is going to be a larger contrast difference between the default theme and the monochromatic themes.

The blue active tab bar is supposed to help address this.
Flags: needinfo?(shorlander)
(In reply to Stephen Horlander [:shorlander] from comment #3)
> No, there is a full color stop between the background and foreground color.

Do you mean it's one step in the photon color palette? Is this a meaningful indicator that there's sufficient contrast? Interestingly, http://design.firefox.com/photon/visuals/color.html#dark-theme uses Grey 90 vs. Grey 70, i.e. two steps? compacttheme.inc.css seems to use something darker than Gray 70.

> The blue active tab bar is supposed to help address this.

Well, it's there, but if this is supposed to be the primary indicator for the active tab, it should probably be more visible?

I just checked on https://webaim.org/resources/contrastchecker/ and #0A84FF vs. #323234 fails most tests.

Case in point: I (not color blind, a bit shortsighted) happen to have had my screen's brightness reduced and the colors shifted to orange (gnome "Night Light" mode). Usually that's not a problem for me, but when I opened the screenshot that made it look particularly bad.
Flags: needinfo?(shorlander)
@stephen, the blue color is too thin to recognize. I am on linux and the slight grey and black contrast are just too not helpful.
(In reply to Stephen Horlander [:shorlander] from comment #3)
> No, there is a full color stop between the background and foreground color.
> There is going to be a larger contrast difference between the default theme
> and the monochromatic themes.
> 
> The blue active tab bar is supposed to help address this.

Is "monochromatic themes" referring to the Dark theme available? Because if so, then I disagree: there is a full color stop, but there is clearly not enough contrast, and the blue active tab bar is just below the blue title bar of the application, so it is very easy to miss that as well.

I don't have any vision problem, and yet, on the shared screenshot, it takes me enough time to determine which tab is the active one to suggest we have room for improvement.
+1; this is my main gripe with Chrome’s Incognito mode: I tab-surf quite intensely, and the low contrast between the foreground tab and the background makes it really hard to spot the active tab. Firefox’s dark theme now has the same problem, and no, the colored line isn’t enough.
Adding this to Photon Follow-up work.
Assignee: nobody → shorlander
Flags: needinfo?(shorlander)
Whiteboard: [photon-followup]
When can we expect some news ?
Flags: needinfo?(shorlander)
For the record, here is a small WebExtension allowing for a starker contrast (red being a sheer example) between the active tab and the others: 
- XPI: https://framadrop.org/r/uKibg6LSGr#Ztmlubq6uAbCRPfsCOQWWAP4d/Q0EsVrSidO3kB7xD8= (available for 60 days)
- gist: https://gist.github.com/SphinxKnight/904ba7a2f1e5e810e6d8dcc1de6e9f9d
Flags: needinfo?(shorlander)
Can we get some news ?
Flags: needinfo?(shorlander)
Julien, can you share that on AMO ?
Flags: needinfo?(sphinx_knight)
Sure, will do in the coming days (under 4 days).
Here is the AMO page hosting the addon https://addons.mozilla.org/fr/firefox/addon/activehl/ (source code identical to #11)
Flags: needinfo?(sphinx_knight)

Probably invalid now with Proton.

Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → INVALID
Flags: needinfo?(stephen)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: