Web Inspector: Pressing Command-Enter should re-evaluate selected console user command

• Component: Web Inspector
• Status: RESOLVED FIXED

[Nikita Vasilyev]

Created attachment 258792 [details]
[Animated GIF] Option-Click to re-run

See the attached GIF.

Note that we don't attach Math.random() row on every click because it matches the last user command. However, in the following example:

    > Math.random()
    < 0.4242424242
    > document.location.pathname
    < "/"

clicking Math.random() should attach it again, e.g.:

    > Math.random()
    < 0.4242424242
    > document.location.pathname
    < "/"
    > Math.random()
    > 0.2354678908

Also, Command+Enter, Option-Enter, or both should re-evaluate selected console message.

[Timothy Hatcher]

I like!  I agree Command or Option-Enter on the selected row should do this too.

[Timothy Hatcher]

Or maybe just Enter for selected row?

[Timothy Hatcher]

Lets also make the double arrow that appears more like the existing grey arrow. So same size/shape but double and blue.

[Radar WebKit Bug Importer]

<rdar://problem/22300296>

[Nikita Vasilyev]

Created attachment 259115 [details]
Patch

[Nikita Vasilyev]

Created attachment 259116 [details]
[Animated GIF] Re-submit on click

I implemented re-submit on click and run it for a day. It turned out not as useful as I have thought because:
1. Console messages shift up every time a new one is added — I need to move mouse cursor after every click.
2. Console always scrolls to the bottom when the new user command result is added, as it should be, but it often leaves the clicked user command out of the view port.

In the attached patch I only implemented re-submit on Enter. I want to give Option-Click to re-run a bit more thought and not include it in this patch.

[Nikita Vasilyev]

Created attachment 259117 [details]
[Animated GIF] With the patch applied

[Devin Rousso]

Comment on attachment 259115 [details]
Patch

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

> Source/WebInspectorUI/UserInterface/Views/LogContentView.js:828
> +        var message = this._selectedMessages[0];

Let instead of var.

> Source/WebInspectorUI/UserInterface/Views/LogContentView.js:839
> +            if (consoleItemElement.__commandView) {

We should change this from "__" to a symbol (can be a different patch, but should be done).

> Source/WebInspectorUI/UserInterface/Views/LogContentView.js:861
> +            WebInspector.logManager.dispatchEventToListeners(WebInspector.LogManager.Event.MessageAdded, {message: commandResultMessage});

Won't this event be triggered once the new command has been executed and added to the frontend, or is it necessary to fire MessageAdded before the message is actually added?

[Nikita Vasilyev]

Comment on attachment 259115 [details]
Patch

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

>> Source/WebInspectorUI/UserInterface/Views/LogContentView.js:861
>> +            WebInspector.logManager.dispatchEventToListeners(WebInspector.LogManager.Event.MessageAdded, {message: commandResultMessage});
> 
> Won't this event be triggered once the new command has been executed and added to the frontend, or is it necessary to fire MessageAdded before the message is actually added?

Removing this line would prevent the result from being added to the console.
It looks like I could use this._logViewController.appendConsoleMessage instead.

[Nikita Vasilyev]

Created attachment 259237 [details]
Patch

[Nikita Vasilyev]

Comment on attachment 259237 [details]
Patch

Since https://bugs.webkit.org/show_bug.cgi?id=148121, pressing Enter when a console message is selected behaves the same way as pressing Enter in the console prompt. I think this is right.

We should be consistent with "Web Inspector: Option-Enter should commit console command without erasing the prompt" https://bugs.webkit.org/show_bug.cgi?id=148123 and use the same shortcut. I used Option–Enter there but I'm open to suggestions.

[Nikita Vasilyev]

Created attachment 259363 [details]
Patch

Rebaselined and significantly simplified.

[WebKit Commit Bot]

Comment on attachment 259363 [details]
Patch

Clearing flags on attachment: 259363

Committed r188637: <http://trac.webkit.org/changeset/188637>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.