[Webkit-unassigned] [Bug 165223] Web Inspector: JS PrettyPrinting - multiline if conditions should be indented

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=165223

--- Comment #3 from Joseph Pecoraro <joepeck at webkit.org> ---
(In reply to comment #2)
> IMO, this would be perfect for an item in the Settings tab.  Personally, I
> write my code to look exactly like the actual output, so I'd like to have
> the option to keep it like that.  Maybe something along the lines of "Match
> Indent of Multi-Line if with First Line" ¯\_(ツ)_/¯

This would be too obscure on its own. Potentially if we had settings for other editing / formatting behaviors then it could be included in a group.

This formatting suggestion matches CodeMirror behavior, and WebKit style <https://webkit.org/code-style-guidelines/#indentation-wrap-bool-op> which is what we typically make our pretty printer do.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.webkit.org/pipermail/webkit-unassigned/attachments/20161130/63372359/attachment.html>