The proton address bar has a grey border with the default theme enabled
Categories
(Firefox :: Address Bar, defect, P1)
Tracking
()
Tracking | Status | |
---|---|---|
firefox86 | --- | unaffected |
firefox87 | --- | unaffected |
firefox88 | --- | unaffected |
firefox89 | --- | affected |
People
(Reporter: cbaica, Unassigned)
References
(Blocks 1 open bug)
Details
(Whiteboard: [proton-address-bar])
Attachments
(1 file)
37.60 KB,
image/png
|
Details |
Affected versions
- Fx89.0a1
Affected platforms
- Ubuntu 18.04
Preconditions
The following prefs are enabled in about:config
- browser.proton.enabled true
- browser.proton.toolbar.enabled
- browser.proton.urlbar.enabled true
Steps to reproduce
- Launch Firefox.
- Focus something in page and observer the address bar.
- Focus the address bar and input a term (e.g. 'cat') so the address bar is expanded with search terms.
Expected result
- The address bar should not have a border.
Actual result
- The address bar has a grey border.
Regression range
- This is not a regression as it's related to a new proton design.
Additional notes
- The address bar does not have a border if either the Proton Light or Proton Dark themes are enabled.
- Firefox has the default theme enabled.
- The same grey border is also visible in Windows 10, if the user has 'Dark' chosen in his OS colors.
Reporter | ||
Updated•3 years ago
|
Updated•3 years ago
|
Comment 1•3 years ago
|
||
By default we respect system colors on Linux, I think the border is from this rule:
https://searchfox.org/mozilla-central/rev/6309f663e7396e957138704f7ae7254c92f52f43/browser/themes/linux/browser.css#192-195
I wonder if we'd want to omit it for design reasons. While it may not be strictly necessary, we should then be confident there will be enough contrast between the toolbar and the input field.
Comment 2•3 years ago
|
||
(In reply to Marco Bonardo [:mak] from comment #1)
By default we respect system colors on Linux, I think the border is from this rule:
https://searchfox.org/mozilla-central/rev/6309f663e7396e957138704f7ae7254c92f52f43/browser/themes/linux/browser.css#192-195I wonder if we'd want to omit it for design reasons. While it may not be strictly necessary, we should then be confident there will be enough contrast between the toolbar and the input field.
I think we should but NI_ing Amy here for validation.
Comment 3•3 years ago
|
||
yes, but it's not just a design decision, there are technical reasons for which Linux doesn't strictly follow the design, mostly the variety of Linux managers themes, and since we don't force the colors on toolbars and fields we may not have enough contrast without a border.
Dao, what do you think?
Updated•3 years ago
|
Updated•3 years ago
|
Comment 4•3 years ago
|
||
Hi,
If this is specifically for Linux cases and it sounds like we don't have control over the toolbar colours I think it's safest to have the boarder for Linux for accessibility.
Comment 5•3 years ago
|
||
Thanks, I'll then resolve as "Won't Fix"
Updated•7 months ago
|
Description
•