[e10s] style on <option> elements isn't respected on Linux (i.e. we need to re-enable dom.forms.select.customstyling for Linux)
Categories
(Core :: Layout: Form Controls, defect, P3)
Tracking
()
People
(Reporter: ossman, Unassigned)
References
(Blocks 1 open bug)
Details
Updated•8 years ago
|
Updated•8 years ago
|
Comment 1•8 years ago
|
||
Reporter | ||
Comment 2•8 years ago
|
||
Reporter | ||
Comment 3•8 years ago
|
||
Updated•8 years ago
|
Comment 4•8 years ago
|
||
Comment 5•8 years ago
|
||
Comment 6•8 years ago
|
||
Comment 9•4 years ago
|
||
(In reply to Daniel Holbert [:dholbert] from comment #6)
At a minimum, we need to sort out the issues described in bug 1338283 before
we can reenable this pref. (I just posted there to see what the status is,
but I'm guessing it's not a top priority at the moment.)
It looks like bug 1338283 has gone away. Emilio, do you think we can land this pref-flip now?
Comment 10•4 years ago
|
||
From running with it enabled for a while in the past, one thing that's very noticeable is the padding in the menuitems being too small.
It goes from depending on the gtk theme to be 1px 5px
, and that's too little IMO (at least compared with the usual GTK themes). So I think at least that bit (in menu.css
) needs tweaking before enabling by default (probably at least making it match Adwaita?)...
With that I think that'd be ok, though maybe we could check with UX or so? But given we support it on macOS and Windows yeah, it seems not very objectionable.
Comment 11•4 years ago
|
||
(In reply to Emilio Cobos Álvarez (:emilio) from comment #10)
From running with it enabled for a while in the past, one thing that's very noticeable is the padding in the menuitems being too small.
Thanks. I've filed bug 1751644 for what I think (?) is that issue.
Comment 12•3 years ago
|
||
Daniel and Emilio, do you think we're at a point where we can re-enable this? Are there outstanding issues we know about that should be added to the dependency list here?
Comment 13•3 years ago
•
|
||
I don't think there are outstanding issues, but on the other hand, we disabled it on macOS. A couple subtests in browser_selectpopup.js fail on Linux (locally, at least), need to debug why too, but I haven't had the time.
Comment 14•3 years ago
•
|
||
I think we could try to enable this (once we've addressed, or annotated [if we must] the test-failures that Emilio noted). I've been running Linux Nightly with the pref enabled for months, and haven't noticed any issues, FWIW.
Comment 15•3 years ago
|
||
There's also a weird menu padding problem with Linux select elements, that doesn't exist in Windows:
Bug 1783498
The people who use transparent background for select elements (for example, to make a site background image visible behind the select) would get a white padding above and below select options in the menu (Example 1 in the bug).
And if they also use a hover background color and a background transition on the select element, the padding background color would switch between white and the hover background color when the mouse cursor leaves or enters the menu/select boundaries (Example 2 in the bug).
Comment 16•3 years ago
|
||
(In reply to importu from comment #15)
There's also a weird menu padding problem with Linux select elements, that doesn't exist in Windows:
I see that same issue in firefox on Windows 10, FWIW; see my screenshot/screencast in bug Bug 1783498 comment 6 - 7. We can discuss/diagnose more on that bug (maybe there's some nuance that I'm missing, or maybe certain Windows configurations are affected vs. unaffected, or maybe the testcase needs something more in order to demonstrate the Windows/Linux difference?)
But at this point, it's not obvious that there's any Linux-specific badness there.
Updated•3 years ago
|
Reporter | ||
Comment 17•2 years ago
|
||
Something has regressed here. The setting dom.forms.select.customstyling
no longer has any effect. :/
Was this intentional?
Comment 18•2 years ago
|
||
(In reply to Pierre Ossman from comment #17)
Something has regressed here. The setting
dom.forms.select.customstyling
no longer has any effect. :/
Hmm, it works as-expected on my end. If I toggle that pref in Firefox Nightly, then e.g. the dropdown-menu on https://bug1783498.bmoattachments.org/attachment.cgi?id=9290740 renders with a blue background behind every menu-option. I tested on Wayland and non-Wayland, using Ubuntu 22.04.
Maybe file a bug on whatever you're seeing, with more details about your configuration/distro and the testcase you're using? (RE changes-that-might-be-involved-here: we landed bug 1828413 in April 2023, which did change the look if that testcase a little bit for me, but the custom colors are still there.)
Reporter | ||
Comment 19•2 years ago
|
||
Never mind, bad testing on my part. Sorry for the noise.
I am seeing a detail I haven't noticed before, and I'm uncertain if it is covered by existing bugs?
Styles applied by the selector option:checked
do not work. option[selected=selected]
does, though. Inspector thinks both match, but the rendering is only affected by the latter. Chrome does not have this issue.
Comment 20•2 years ago
|
||
No worries. I spun off bug 1856725 for the option:checked
thing; I see that too.
Comment hidden (advocacy) |
Comment 23•1 year ago
•
|
||
James, continued comments like comment 22 won't be tolerated; please read https://bugzilla.mozilla.org/page.cgi?id=etiquette.html , particularly "No abusing people.", and especially this second part:
- No obligation. "Open Source" is not the same as "the developers must do my bidding." Everyone here wants to help, but no one else has any obligation to fix the bugs you want fixed. Therefore, you should not act as if you expect someone to fix a bug by a particular date or release. Aggressive or repeated demands will not be received well and will almost certainly diminish the impact of and interest in your suggestions.
Keep in mind, Firefox is a product offered to users for free, developed by a relatively small team, juggling many bugs and many features across many platforms. Some bugs are higher priority than others; some bugs are more complex than others (even if they may not look like it on the surface). This does mean that some bugs sit open for many years.
Bear in mind that I do not have control of what browsers or operating systems my customers use.
That's true. You might be alarmed to learn that Safari every browser on macOS (Chrome/Safari/Firefox), every browser on iOS / iPadOS, Chromium-on-Android (along with Firefox on Android) all render your menu-dropdown with a plain unstyled background, just like Firefox-on-Linux does.
What is the point of closing a problem as a duplicate of another problem if ther other problem is not resolved.
This is standard practice for our bug tracking system (and many others). It's not meant as an affront to the duplicated bug; it's simply so that we don't end up with many different bug reports with different people repeating work in different places.
NOBODY HAD EVEN LOOKED AT THIS ISSUE IN SEVEN YEARS!
That's not at all true; it's made slow progress over the years, and has been looked at. The most recent three comments are from 7 months ago; and each of the 6 "Depends-on" bugs (which have been fixed) were each stepping stones towards getting this fixed.
Anyway: this was a courtesy response; further engagement along the lines of comment 22 are unacceptable and unwelcome.
Description
•