Open
Bug 1542734
Opened 5 years ago
Updated 2 years ago
Rule view auto-corrects named CSS colors with mixed case to lowercase
Categories
(DevTools :: Inspector: Rules, defect, P3)
DevTools
Inspector: Rules
Tracking
(firefox66 wontfix, firefox67 wontfix, firefox68 fix-optional)
NEW
Tracking | Status | |
---|---|---|
firefox66 | --- | wontfix |
firefox67 | --- | wontfix |
firefox68 | --- | fix-optional |
People
(Reporter: cfogel, Unassigned)
References
(Regression)
Details
(Keywords: regression)
Attachments
(1 file)
4.80 KB,
image/png
|
Details |
Affected versions
- 66.0.2, 67.0b8, 68.0a1 (2019-04-07);
Affected platforms
- Windows 10, macOS 10.14, Ubuntu 16.04
Steps to reproduce
- Launch Firefox, enable the devTools inspector with the Rules section and Changes tab being displayed;
- In the rules inspector, for any rule type the following line:
color: rEd; - Press the Enter key;
Expected result
- the text format should remain the same (color: rEd;);
Actual result
- the text format is all lowercase letters (color: red;);
Regression range
- First bad build_date: 2018-12-06 09:33:31.527000 changeset: eeddd827762d17b290ffe65d17baf6196477f488
- Last good build_date: 2018-12-06 06:33:30.298000 changeset: ddd8bdd2f05e27159720709f8dd40a7d9dd006f6
- Pushlog: link
- Potential regressor: bug 1284852;
Additional notes
- attache screenshot with the issue;
- the Changes tab keeps the formatting as-is;
Reporter | ||
Comment 1•5 years ago
|
||
Hey Patrick, mind taking a look at this and confirming if it's the case?
Thanks!
Has Regression Range: --- → yes
Has STR: --- → yes
Flags: needinfo?(pbrosset)
Comment 2•5 years ago
|
||
Yes I can confirm the bug. Indeed we do want to show "authored" styles in the Rules view as much as we can, so we should preserve the mixed case here. The regressor seems likely.
The Changes panel just listens to changes being made in the rule-view so ends up storing the right spelling. But the Rules view transforms the value because it's a color, so a special one. The code that does this likely converts color names.
Adjusting the tracking flags. I don't think this is bad enough that we want to attempt an uplift.
Updated•5 years ago
|
Priority: -- → P3
Summary: Rule color-values input as lowercase autocomplete even if characters are mixed instead of as-is → Rule view auto-corrects named CSS colors with mixed case to lowercase
Updated•5 years ago
|
Updated•2 years ago
|
Severity: minor → S4
You need to log in
before you can comment on or make changes to this bug.
Description
•