Closed Bug 1215373 Opened 10 years ago Closed 9 years ago

[rule-view] Add support for ::backdrop pseudo-element

Categories

(DevTools :: Inspector: Rules, defect)

Product:
DevTools
Component:
Inspector: Rules
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(firefox44 affected)

Status:
RESOLVED FIXED
Tracking Flags:
Tracking Status
firefox44 --- affected

People

(Reporter: xidorn, Unassigned)

References

(Blocks 1 open bug)

Details

We are adding ::backdrop pseudo-element for fullscreen element in bug 1064843. The devtools should consider adding support of it.
We have a list of pseudo-elements that the inspector supports here: https://dxr.mozilla.org/mozilla-central/source/devtools/shared/styleinspector/css-logic.js#46. In theory, adding it to that set will "just work", as those are passed into DOMUtils.getCSSStyleRules: https://dxr.mozilla.org/mozilla-central/source/layout/inspector/inDOMUtils.cpp#213.
Summary: Add support for ::backdrop pseudo-element → [rule-view] Add support for ::backdrop pseudo-element
See also bug 1216234 - I think this is a good opportunity to future-proof the list by moving it to domutils.
See Also: → 1216234
I suspect this should just work now that bug 1216234 landed.
That looks promising. I'll close this bug if that works for ::backdrop.
Component: Developer Tools: Inspector → Developer Tools: CSS Rules Inspector
Yeah, looks like it works now, although the highlighting elements function doesn't really work with this pseudo element. That could be in other bug I suppose.
Status: NEW → RESOLVED
Closed: 9 years ago
Depends on: 1216234
Resolution: --- → FIXED
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.