950090 - [ruleview] It is very easy to accidentally change properties of a pseudo-element instead of the element

Status: RESOLVED FIXED

(DevTools :: Inspector, defect)

Description

[Jared Wein [:jaws] (please needinfo? me)]

The rule view now allows changing properties of an element's associated pseudo-elements, but it is also very easy to spend time changing properties only to realize a little bit later that your changes aren't showing up because you've been changing properties of the pseudo-element instead of the actual element.

I don't have any great ideas on how to fix this, but it is a problem that both :paul and I have encountered.

Comment 1

[Brian Grinstead [:bgrins]]

(In reply to Jared Wein [:jaws] from comment #0)
> The rule view now allows changing properties of an element's associated
> pseudo-elements, but it is also very easy to spend time changing properties
> only to realize a little bit later that your changes aren't showing up
> because you've been changing properties of the pseudo-element instead of the
> actual element.
> 
> I don't have any great ideas on how to fix this, but it is a problem that
> both :paul and I have encountered.

We could simply move the pseudo element rules to the bottom of the rule view, though we decided to put it at the top originally for fear of them not being discovered.  I would prefer to wait to move them down to the bottom until after we allow proper selection in the markup tree (Bug 920141).  But we could consider doing this or something else I haven't thought of with the UI beforehand.

Comment 2

[Brian Grinstead [:bgrins]]

This should be fixed now that Bug 920141 has landed and ::before/::after are no longer shown in the rule view, but instead in the markup view.