Bug 16349 – Double-clicking styles in the Computed Style or User Agent sections to edit them silently fails

• Component: Web Inspector (Deprecated)
• Status: RESOLVED WONTFIX

[Adam Roben (:aroben)]

Double-clicking styles in the Computed Style section to edit them silently fails. I'm guessing that this is the source of a lot of the "inline css editing doesn't work" comments on <http://webkit.org/blog/148/web-inspector-update/>.

One possible solution is to make double-clicking a property in the Computed Style section copy that property to the Inline Style Attribute section and start editing it there. I think it's important that double-clicking in the Computed Style section let you edit *somewhere*, rather than popping up an error dialog or something.

[Timothy Hatcher]

The same happens for user agent rules and mapped attributes. We could allow editing those i think, but it seems weird.

[Timothy Hatcher]

*** Bug 16350 has been marked as a duplicate of this bug. ***

[fALk Gärtner]

For me editing also does not work on the inline element. no matter where you click I can only select values (for copying) but I can not change any of them. maybe there is something more fundamental missing on my part, but it sure as hell is no "click and play" at the moment ;) i double clicked everywhere tried every ctrl-click and there is just nowhere an edit value box as shown on the blog. 
Also I would love to click in the metrix box to change box models if that is possible. Just let me change whatever value I defined in the first place wherever it appears - in the code/stylesheet, in the stlye, metrics and property boxes - everywhere. would make life as a part time webdeveloper so much easier. 

[Adam Roben (:aroben)]

(In reply to comment #3)
> Also I would love to click in the metrix box to change box models if that is
> possible. Just let me change whatever value I defined in the first place
> wherever it appears - in the code/stylesheet, in the stlye, metrics and
> property boxes - everywhere. would make life as a part time webdeveloper so
> much easier. 

Could you file a new bug about this enhancement request?

[Adam Roben (:aroben)]

<rdar://problem/5712879>

[Wout Mertens]

the Firebug editor for Mozilla does allow adding ad-hoc css rules for an element.

Maybe double-clicking a computed style should create a new style box specific to the element and copy the attribute name and value into it, allowing the user to edit that.

[Timothy Hatcher]

*** Bug 21325 has been marked as a duplicate of this bug. ***

[Timothy Hatcher]

*** Bug 21488 has been marked as a duplicate of this bug. ***

[Joseph Pecoraro]

Is this something that we want to add or should Computed / User Agent / Mapped Attribute styles be off limits to editing?  There are already plenty of other ways to add styles.

If we keep them uneditable then there should be some indication to the user (as evident by all the duplicates of this bug).

[Timothy Hatcher]

(In reply to comment #9)
> Is this something that we want to add or should Computed / User Agent / Mapped
> Attribute styles be off limits to editing?  There are already plenty of other
> ways to add styles.
> 
> If we keep them uneditable then there should be some indication to the user (as
> evident by all the duplicates of this bug).

I think double click could copy the property to the Style Attribute section and edit.

We should also show non-editable sections differently. Maybe a "not pencel" icon in the section header like Finder uses for folders?

[Alexander Pavlov (apavlov)]

(In reply to comment #10)
> (In reply to comment #9)
> > Is this something that we want to add or should Computed / User Agent / Mapped
> > Attribute styles be off limits to editing?  There are already plenty of other
> > ways to add styles.
> > 
> > If we keep them uneditable then there should be some indication to the user (as
> > evident by all the duplicates of this bug).
> 
> I think double click could copy the property to the Style Attribute section and edit.
> 
> We should also show non-editable sections differently. Maybe a "not pencel" icon in the section header like Finder uses for folders?

Currently, we mark non-editable sections with a gray background which makes it quite clear that they are read-only. Does this sound good enough wrt the rw/ro distinction? Are we still bothered with letting the user edit at least something when they double-click inside the read-only section?

[Alexander Pavlov (apavlov)]

The currently implemented graying-out seems to be a good and intuitively clear indicator of read-only sections.