9

Devtools Style editor - Style sheet could not be loaded [ x ] close button has w...

 3 years ago
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.
neoserver,ios ssh client
Closed Bug 1701562 Opened 2 months ago Closed 26 days ago

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)

RESOLVED FIXED

90 Branch

Tracking Status firefox-esr78 --- unaffected firefox88 --- wontfix firefox89 --- wontfix firefox90 --- fixed

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;
  1. Launch Firefox, enable DevTools set it to dark theme;
  2. Open the Stylesheet tab;
  3. 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.
Has Regression Range: --- → no
Has STR: --- → yes

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

Blocks: 1701622
Priority: -- → P3

Cristian, the description is talking about "the eye icon appears pixelated or deformed at times while loading" also in bug 1701555.
Cloned description?

Flags: needinfo?(cristian.fogel)

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.

Flags: needinfo?(cristian.fogel)

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.

QA Whiteboard: [qa-regression-triage]

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

Keywords: good-first-bug

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.

Assignee: nobody → gpr
Status: NEW → ASSIGNED

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.

Flags: needinfo?(odvarko)

Julian, can you please help here, thank you!

Flags: needinfo?(odvarko) → needinfo?(jdescottes)

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.

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

About Joyk


Aggregate valuable and interesting links.
Joyk means Joy of geeK