Closed Bug 1863006 Opened 2 years ago Closed 2 years ago

Display selector warnings in Rules view

Categories

(DevTools :: Inspector: Rules, enhancement, P2)

enhancement

Tracking

(firefox121 fixed)

RESOLVED FIXED
121 Branch
Tracking Status
firefox121 --- fixed

People

(Reporter: nchevobbe, Assigned: nchevobbe)

References

(Regressed 1 open bug)

Details

Attachments

(3 files)

No description provided.

MDN might have dedicated pages for the issue (e.g. unconstrained :has perf warning might be documented https://github.com/mdn/mdn/issues/469)

This will be used to have a common class we can target between the rule selector
and the ones in ancestor data (next patch in queue)

We used to return selectorText for those, but we want to have individual elements
for each selectors so we can put selector warnings icon next to the right selector.

Depends on D192864

For now this only handles UnconstrainedHas warnings, but we may have more
in the future.

Depends on D192865

Attachment #9362171 - Attachment description: Bug 1863006 - [devtools] Add `ruleview-selector-container` class on selector element. r=#devtools-reviewers. → Bug 1863006 - [devtools] Tweak css classes for rules view selectors. r=#devtools-reviewers.
Pushed by nchevobbe@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/cc98109f1857 [devtools] Tweak css classes for rules view selectors. r=devtools-reviewers,ochameau. https://hg.mozilla.org/integration/autoland/rev/c8b60425f0a2 [devtools] Return selectors array for ancestor rules. r=devtools-reviewers,ochameau. https://hg.mozilla.org/integration/autoland/rev/a4657a05cffd [devtools] Display selector warnings in Rule view. r=devtools-reviewers,fluent-reviewers,ochameau.
Regressions: 1864752
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 121 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: