Web Inspector: Make the first click on a rule section create a newline for easy property addition

[Devin Rousso]

Created attachment 255890 [details]
Expected Functionality (from Chrome)

Clicking on the last line of a rule section should create a newline and place the cursor on the beginning of that newline to make adding new properties easier.  This should only happen, however, on the first click of that rule section, meaning that once it is focused (and until it has lost focus) clicking on the last line will simply put the cursor at the end of that line.

[Radar WebKit Bug Importer]

<rdar://problem/21625881>

[Devin Rousso]

Created attachment 255893 [details]
Patch

[Devin Rousso]

Created attachment 255894 [details]
After Patch is applied

[Timothy Hatcher]

Comment on attachment 255893 [details]
Patch

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

> Source/WebInspectorUI/ChangeLog:12
> +        (WebInspector.CSSStyleDeclarationTextEditor.prototype._editorFocused): When the editor gains focus and the cursor is on
> +        the last line of the rule, insert a new line and place the cursor on that line.

Chrome seems to do this on any line. Should we? I think last line is fine too.

[Devin Rousso]

Created attachment 255942 [details]
Patch

[Timothy Hatcher]

Comment on attachment 255942 [details]
Patch

Awesome!

[WebKit Commit Bot]

Comment on attachment 255942 [details]
Patch

Clearing flags on attachment: 255942

Committed r186212: <http://trac.webkit.org/changeset/186212>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.