Devtools Style editor - Style sheet could not be loaded [ x ] close button has w...
source link: https://bugzilla.mozilla.org/show_bug.cgi?id=1701562
Go to the source link to view the article. You can view the picture content, updated content and better typesetting reading experience. If the link is broken, please click the button below to view the snapshot at that time.
Devtools Style editor - Style sheet could not be loaded [ x ] close button has wrong color with dark theme
Categories
(DevTools :: Style Editor, defect, P3)
Tracking
(firefox-esr78 unaffected, firefox88 wontfix, firefox89 wontfix, firefox90 fixed)
90 Branch
People
(Reporter: cfogel, Assigned: gpr)
References
(Blocks 1 open bug, Regression)
Details
(Keywords: good-first-bug, regression)
Affected versions
- 88.0a1(2021-03-25), 87.0b2
Affected platforms
- Windows 10;
Steps to reproduce
- not sure how to trigger it consistently, happened slow-loading internet spikes;
- Launch Firefox, enable DevTools set it to dark theme;
- Open the Stylesheet tab;
- Load any webpage;
Expected result
- Stylesheet error - [ x ] close button is visible;
Actual result
- with the current color scheme the [ x ] close button is not visible on Dark Theme (dark grey background + black icon);
Regression range
- cannot get a proper regression range until the message can be consistently triggered;
- Last good: 2019-09-27 build that I managed to confirm;
Additional notes
- attached "good version" screenshot;
- attached recording to best illustrate the issue on the next comment.
Thank you Cristian for the report, I can reproduce it on my machine Win10 + Fx Nightly
Edit: the description has changed, not clear how to reproduce now.
Honza
Cristian, the description is talking about "the eye icon appears pixelated or deformed at times while loading" also in bug 1701555.
Cloned description?
Indeed, cloned and jumped over the section when updating data for this bug.
Thanks Honza for pointing it out, have updated the initial comment with correct description.
Is there any page I could load to see the "Stylesheet could not be loaded" error?
With the initial report it was noticed it on https://www.stirileprotv.ro, https://www.bbc.com/and some links from the Pocket-newtab.
Nothing consistent and reproducible 100% of the time though.
Attaching a test case found here as well as regression range made on Windows 10x64. It seems that the test case must be saved locally in order to reproduce the issue.
Last good revision: 5e43a93a9dc5eac61977cd87cf45d8d27565dcf5
First bad revision: c9a470a98696c8524f73d97d0255abc2a52ffa62
Pushlog:
https://hg.mozilla.org/integration/autoland/pushloghtml?fromchange=5e43a93a9dc5eac61977cd87cf45d8d27565dcf5&tochange=c9a470a98696c8524f73d97d0255abc2a52ffa62
Thank you Alexandru!
Note that the test case in comment #7 needs to be loaded through the file: protocol
This change might caused the problem. We should look at what CSS was broken by that.
https://hg.mozilla.org/integration/autoland/rev/8dae0f4af8f85654c68b853133d658aacdb9ea20
Honza
Hello, I would like to work on this bug.
Thank you for the help Gagah.
Please don't forget to look at the comment #8 and see why this bug happened in the first place.
So the regression happens because messageCloseButton
is changed from div
to button
. It makes currentColor
in messageCloseButton
css using inherit color
from default firefox button style.
The fix is pretty simple, just add color: inherit
or color: currentColor
for messageCloseButton
css to explicitly inherit or use color
from its parent.
The next problem is background-color
for .messageCloseButton:hover
is hardcoded to gray
. What color/variable should I use to change this background-color
?
Hello, I still need information about background-color
for .messageCloseButton:hover
because currently it's hardcorded to gray
and the background color is blending with the cross color.
Julian, can you please help here, thank you!
Thanks for the information! Now I can continue to work on the fix.
We can let it ride the trains, it's a minor visual nit and it regressed in 85.
Recommend
About Joyk
Aggregate valuable and interesting links.
Joyk means Joy of geeK