Bug 143814 – Web Inspector: Allow toggling the edibility of a DOMTreeOutline

• Status: RESOLVED
• Resolution: FIXED
• Component: Web Inspector

[Joseph Pecoraro]

* SUMMARY
Allow toggling the edibility of a DOMTreeOutline.

A DOMTreeOutline in the console should not be editable. You can "Reveal in DOM Tree" to edit from the DOM tree. ObjectTree's themselves are not editable right now.
A DOMTreeContentView should always have editable DOMTreeOutlines.

[Radar WebKit Bug Importer]

<rdar://problem/20565653>

[Joseph Pecoraro]

Created attachment 250896 [details]
[PATCH] Proposed Fix

[Joseph Pecoraro]

Patch doesn't apply because it relies on an earlier change in very similar code.

[Brian Burg]

Comment on attachment 250896 [details]
[PATCH] Proposed Fix

View in context: https://bugs.webkit.org/attachment.cgi?id=250896&action=review

Very interesting, I made nearly the exact same change in my branch. (Snapshotted DOM nodes aren't editable either.)

It seems you missed DOMTreeElement._canAddAttributes (set in constructor) and disabling draggability (DOM elements are configured in onattach). Once these cases are fixed this patch should be ready to go.

> Source/WebInspectorUI/UserInterface/Views/DOMTreeOutline.js:146
> +    set editable(x)

I would prefer 'value' rather than 'x'.

[Joseph Pecoraro]

(In reply to comment #4)
> Comment on attachment 250896 [details]
> [PATCH] Proposed Fix
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=250896&action=review
> 
> Very interesting, I made nearly the exact same change in my branch.
> (Snapshotted DOM nodes aren't editable either.)
> 
> It seems you missed DOMTreeElement._canAddAttributes (set in constructor)
> and disabling draggability (DOM elements are configured in onattach). Once
> these cases are fixed this patch should be ready to go.

Great catches!

[Joseph Pecoraro]

> It seems you missed DOMTreeElement._canAddAttributes (set in constructor)

This seems like it should stay unaffected. This essentially distinguishes that this DOMTreeElement is the opening tag of an element, so if it was editable attributes can be added. Perhaps it should be renamed.

> and disabling draggability (DOM elements are configured in onattach).

Yep!

[Joseph Pecoraro]

Created attachment 250943 [details]
[PATCH] Proposed Fix

[Joseph Pecoraro]

> > Source/WebInspectorUI/UserInterface/Views/DOMTreeOutline.js:146
> > +    set editable(x)
> 
> I would prefer 'value' rather than 'x'.

Oops, I forgot about this. We tend to use x pretty consistently though. Especially for short functions.

[Brian Burg]

Comment on attachment 250943 [details]
[PATCH] Proposed Fix

r=me

[Joseph Pecoraro]

http://trac.webkit.org/changeset/182905