Bug 115987
| Summary: | think about how auto-NLs in console can be improved | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Paul Miller <paul+bugs> |
| Component: | Web Inspector | Assignee: | Nobody <webkit-unassigned> |
| Status: | RESOLVED FIXED | ||
| Severity: | Normal | CC: | graouts, joepeck, timothy, webkit-bug-importer |
| Priority: | P2 | Keywords: | InRadar |
| Version: | 528+ (Nightly build) | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
Paul Miller
Been trying to use webkit.app as main browser, my main concern is new auto-creation of newlines instead of executing the statement.
When it indeed allows to improve readability, it’s often that devs write some boilerplate code like `list.filter(function() {})` and then write code inside function and press enter.
With current implementation, I need to press additionally ⌘→ to move to the end of current string and execute the command.
Any ideas on how can this be improved? I write a lot of code in inspector every day and this can be annoying (maybe I ain’t used to it yet tho)
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Radar WebKit Bug Importer
<rdar://problem/13870951>
Timothy Hatcher
Command-Return always executes. Does that work for you?
Paul Miller
I guess no as it will also trigger auto-complete (it does currently). Otherwise, yep, probably. Still more annoying than before, but anyway.
In some cases, like strings ('<enter>') the syntax is also invalid.
Timothy Hatcher
(In reply to comment #3)
> I guess no as it will also trigger auto-complete (it does currently). Otherwise, yep, probably. Still more annoying than before, but anyway.
>
> In some cases, like strings ('<enter>') the syntax is also invalid.
It triggering autocomplete is a bug I will fix. Thanks!
You make a good point that enter in the middle of some expressions is not desired and would result in a syntax error. That would be a good improvement. Right now we always insert a newline if the cursor is not at the end of the prompt.
Timothy Hatcher
Was fixed 5/14/13 5:12 PM