Open Bug 1695061 Opened 9 months ago Updated 8 months ago

Consider using currentColor for focus-outline-color when an external lwtheme is applied

Categories

(Firefox :: Theme, task, P3)

task
Points:
1

Tracking

()

People

(Reporter: harry, Unassigned)

References

(Blocks 2 open bugs)

Details

(Whiteboard: [proton-address-bar])

Attachments

(1 file)

Attached image image.png

--focus-outline-color is set based on whether the theme is brighttext or not. This may lead to unexpected results when a theme applied brighttext to :root, but it isn't a dark theme in all places. One example is this theme https://addons.mozilla.org/en-US/firefox/addon/gradientus/, which uses white text in most of the UI, but the Urlbar is darktext. We apply the dark theme focus outline to a light Urlbar.

We could use currentColor for --focus-outline-color when a lwtheme is applied. If that provides enough contrast for text, then it would be adequate for borders too. We would still use #00DDFF for our built-in dark theme.

Themes could still set the address bar border color by defining toolbar_field_border_focus.

I'm marking this as proton-address-bar since the address bar is the most visible example of this issue, but it's an issue in other parts of the theme as well.

I think this is ultimately work that falls under Firefox :: Themes, so putting in that component.

Component: Toolbars and Customization → Theme

How critical is this, considered themes can already set toolbar_field_border_focus, replacing the poor border we may pick in those cases?
I'm mostly wondering whether currentColor wouldn't look more like a normal border than a focus outline.

You need to log in before you can comment on or make changes to this bug.