As discussed offline with Joe, it can be aggravating sometimes to click after the end of a CSS property in the styles sidebar and having a newline be automatically inserted. The same is true of the warning icons, especially for those who are well versed with CSS and understand what the strikethroughs can mean.
Created attachment 303449 [details] Patch
Created attachment 303450 [details] [Image] After Patch is applied
Created attachment 303451 [details] Patch
Created attachment 303452 [details] [Image] After Patch is applied Had to rework a small spacing issue :D
Interesting! We may need to move to multiple columns in the Settings pane, its starting to get longer. I haven't yet reviewed the patch, I want to try it. We don't want to go overboard with settings so I'd like to see what others think (and myself after I try it).
Comment on attachment 303451 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=303451&action=review > Source/WebInspectorUI/Localizations/en.lproj/localizedStrings.js:129 > +localizedStrings["CSS Editing:"] = "CSS Editing:"; The sidebar is "Styles" or "Rules", we don't use CSS in the UI. > Source/WebInspectorUI/Localizations/en.lproj/localizedStrings.js:717 > +localizedStrings["Show warning icons"] = "Show warning icons"; "Show inline warnings". The fact that is an icon is an implementation detail.
Created attachment 303854 [details] Patch So I reworked the "Select text on first click" option to work more like how Chrome/Firefox currently function, meaning that it will always select the text range only on the first click.
Comment on attachment 303854 [details] Patch Clearing flags on attachment: 303854 Committed r213635: <http://trac.webkit.org/changeset/213635>
All reviewed patches have been landed. Closing bug.