Open Bug 1518831 Opened 5 years ago Updated 2 years ago

Invalid declarations inherited from inline styles are removed from Rules view

Categories

(DevTools :: Inspector: Rules, defect, P3)

defect

Tracking

(firefox-esr60 unaffected, firefox64 wontfix, firefox65 fix-optional, firefox66 fix-optional)

Tracking Status
firefox-esr60 --- unaffected
firefox64 --- wontfix
firefox65 --- fix-optional
firefox66 --- fix-optional

People

(Reporter: JuliaC, Unassigned)

References

Details

(Keywords: regression)

[Affected versions]:

  • 65.0b9 build1 (20190107180200)
  • 66.0a1 (2019-01-08)

[Affected platforms]:

  • Windows 10 x64
  • macOs 10.12.6
  • Ubuntu 16.04 x86
  1. Launch Firefox
  2. Go to https://www.youtube.com/
  3. Right click on any element and select the Inspect Element context menu option
  4. For the "element" selector look for the "font-size" (or "font-family") declaration and change the property name in a invalid one
  • inspect the Changes tab
  • inspect the Rules section

[Expected result]:

  • the declarations are disabled and marked as invalid

[Actual result]:

  • the declarations are removed

[Regression range]:

Forgot to attach the screencast for the mentioned issue: https://drive.google.com/file/d/19QERyoYgoq7uh3rdz79cuBoHjHjMjren/view?usp=sharing.
Also, another note: after the affected rules are removed, this fact is NOT reflected in the Changes tab.

Thanks for filing!

Clarification for the person who's going to address the bug: once editing declarations inherited from an inline style attribute (this is what's meant by the "element" selector in the steps to reproduce above), they get removed from the Rules view if the property name is invalid. Invalid declarations inherited from other selectors are not impacted.

Priority: -- → P3
Summary: Some declarations are removed if their property names are invalid → Invalid declarations inherited from inline styles are removed from Rules view

Happy to take a patch in nightly 67, or potentially, in beta 66 for this.
I'm marking it fix-optional to remove it from weekly regression triage, since it has a priority assigned.

Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.