Address bar results have transparent background with certain webextension themes
Categories
(Firefox :: Address Bar, defect, P2)
Tracking
()
Tracking | Status | |
---|---|---|
firefox72 | --- | fixed |
People
(Reporter: mconca, Assigned: dao)
References
Details
Attachments
(2 files)
Starting with Nightly on 03-Oct, typing in the address bar drops it down with a transparent background and the search suggestions centered. The underlying tab content is visible through the drop-down list. See attached image.
STR:
- Pick a tab, any tab
- Click the address bar
- Start typing
System is Windows 10.
Comment 1•5 years ago
|
||
Do you have a userchrome.css?
Reporter | ||
Comment 2•5 years ago
|
||
I do not have a userchrome.css file. However, I've narrowed it down and can verify it only happens when I have a theme installed.
STR
- Create a new profile
- Install this theme from AMO (probably happens with others, too)
- Click inside the address bar
- Start typing
Comment 3•5 years ago
|
||
With "Fall Foggy Forest by MaDonna" theme from AMO, I face the same problem.
Assignee | ||
Updated•5 years ago
|
I am having the same issue, but with the Firefox Color add-on enabled.
Disabling the megabar ( browser.urlbar.megabar = false
in about:config
) returns normal opacity and readability to the address bar.
Comment 5•5 years ago
|
||
Is the mega bar using the lightweight theme value for the URL bar? We're currently letting theme designers set that value as transparent. We might need to break out the dropdown menu from the URL bar to a) prevent the drop down options from becoming transparent and b) allowing theme designers to style it.
Assignee | ||
Comment 6•5 years ago
|
||
(In reply to Caitlin Neiman [:caitmuenster] from comment #5)
Is the mega bar using the lightweight theme value for the URL bar?
Yes.
We're currently letting theme designers set that value as transparent.
Can we stop doing that?
We might need to break out the dropdown menu from the URL bar to a) prevent the drop down options from becoming transparent and b) allowing theme designers to style it.
We don't want that. The dropdown is supposed to visually integrate with the textbox.
Updated•5 years ago
|
Comment 8•5 years ago
|
||
(In reply to Dão Gottwald [::dao] from comment #6)
We're currently letting theme designers set that value as transparent.
Can we stop doing that?
I believe this would take away the possibility to have the background shine through. I'm thinking if it would make more sense to force a non-transparent background in the dropdown instead, is this something you could do? Stopping theme developers from doing that would need time to adjust and communications to affected developers.
Assignee | ||
Comment 9•5 years ago
|
||
The results list isn't a separate popup anymore, it's part of urlbar and thus just uses its background. We could technically set a separate background on the list, but this would kind of defeat the megabar design...
Comment 10•5 years ago
|
||
What I'd like to see is that we use one color for the url bar, taken from the theming value. In case this color is transparent, the url has a split background where the part on top of the url bar is transparent and the remaning area is forced to an opaque color, likely whatever color the popup had before the megabar redesign.
If that is technically feasible then I think we should go this route instead of disallowing transparent backgrounds. While it may defeat the purpose of the megabar being a big popup, I think we should leave theme developers this freedom. We can add recommendations to our documentation on using transparent values.
The use case for having a transparent background is valid, and I'd like avoid the comms and deprecation cycle necessary for this sort of change.
Assignee | ||
Comment 11•5 years ago
|
||
(In reply to Philipp Kewisch [:Fallen] [:📆] from comment #10)
What I'd like to see is that we use one color for the url bar, taken from the theming value. In case this color is transparent, the url has a split background where the part on top of the url bar is transparent and the remaning area is forced to an opaque color, likely whatever color the popup had before the megabar redesign.
Even without any results displayed, the focused megabar overlaps the bottom of the tabs and the top of the content area. This looks pretty broken with a completely transparent background.
Comment 12•5 years ago
|
||
Philipp, are you saying that you'd be prefer to keep the transparency in all urlbar states? I could agree with keeping support for transparency in the unfocused state, but once the user interacts with it so that it's focused (and with the megabar pref on it animates and grows) it's fair game and we take over.
Usability trumps customizability, which is one of the principles which made us decide to delay shipping to Fx 72.
Comment 13•5 years ago
|
||
My suggestion was actually a bit more complicated and involved having a fake background in Firefox that covers the area that goes outside of the toolbar area.
For a simpler solution, looking at the theme color values, what about using toolbar_field
for the unfocused state (which can be transparent), and use popup
for the focused state, for the whole toolbar. I don't think the popup
value can be transparent.
Comment 14•5 years ago
|
||
I could also imagine using toolbar_field_focus
for the focused state, hoping that isn't allowed to be transparent. Caitlin, do you know which colors are allowed to be transparent? I didn't find this anywhere in our documentation.
Assignee | ||
Comment 15•5 years ago
|
||
(In reply to Philipp Kewisch [:Fallen] [:📆] from comment #13)
For a simpler solution, looking at the theme color values, what about using
toolbar_field
for the unfocused state (which can be transparent), and usepopup
for the focused state, for the whole toolbar.
We could do that, except that ideally the address bar shouldn't drastically change colors when focusing it which I guess it will do with many themes if we repurpose popup
like that.
I don't think the
popup
value can be transparent.
I believe it can actually. I don't see us dropping the alpha channel in LightweightThemeConsumer.jsm.
(In reply to Philipp Kewisch [:Fallen] [:📆] from comment #14)
I could also imagine using
toolbar_field_focus
for the focused state, hoping that isn't allowed to be transparent.
That's what we're already using.
Comment 16•5 years ago
|
||
So I'd be interested to learn how many addons use transparent color values for toolbar_field_focus
and which value (between 0
and 1
) they use.
We could also consider to implement a graceful degradation towards only allowing transparency values between 0
and 0.5
- informed by the information we can get from existing addons.
Comment 17•5 years ago
|
||
(In reply to Mike de Boer [:mikedeboer] from comment #16)
So I'd be interested to learn how many addons use transparent color values for
toolbar_field_focus
and which value (between0
and1
) they use.
We could also consider to implement a graceful degradation towards only allowing transparency values between0
and0.5
- informed by the information we can get from existing addons.
Hey Mike, would you be able to provide a regexp for this?
Updated•5 years ago
|
Assignee | ||
Updated•5 years ago
|
Assignee | ||
Comment 18•5 years ago
|
||
Comment 19•5 years ago
|
||
Assignee | ||
Updated•5 years ago
|
Comment 20•5 years ago
|
||
bugherder |
Assignee | ||
Updated•1 year ago
|
Description
•