Bug 110548 - Web Inspector: Console view should allow to clear the prompt in a quicker way
Summary: Web Inspector: Console view should allow to clear the prompt in a quicker way
Status: RESOLVED WONTFIX
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Inspector (Deprecated) (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Vivek Galatage
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-02-21 20:40 PST by Vivek Galatage
Modified: 2013-02-21 21:04 PST (History)
8 users (show)

See Also:

Attachments
Patch (2.42 KB, patch)
2013-02-21 20:43 PST, Vivek Galatage 		vsevik: review-
Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Vivek Galatage 2013-02-21 20:40:48 PST 
When using the console view prompt, many a times developers end up entering some code snippet and then decide to erase it to something else. 

This shortcut provides a way to quickly clear the prompt. Patch follows.
Comment 1 Vivek Galatage 2013-02-21 20:43:50 PST 
Created attachment 189669 [details]
Patch
Comment 2 Eugene Klyuchnikov 2013-02-21 20:58:25 PST 
Comment on attachment 189669 [details]
Patch

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

> Source/WebCore/inspector/front-end/ConsoleView.js:592
> +        var shortcutC = shortcut.makeDescriptor("c", WebInspector.KeyboardShortcut.Modifiers.Ctrl);

Ctrl-C is already bound to copy selected text.
Redefining common shortcut doesn't sound good.

BTM, cleaning prompt with Ctrl-A then Delete or Backspace seems to be rather convenient.
Comment 3 Vsevolod Vlasov 2013-02-21 21:00:37 PST 
Comment on attachment 189669 [details]
Patch

I think "Ctrl/Cmd + A, then Delete" and "Enter" are two shortcuts that already exist for that.
Overriding Ctrl+C's default behavior is also weird.
Comment 4 Vivek Galatage 2013-02-21 21:04:23 PST 
(In reply to comment #3)
> (From update of attachment 189669 [details])
> I think "Ctrl/Cmd + A, then Delete" and "Enter" are two shortcuts that already exist for that.
> Overriding Ctrl+C's default behavior is also weird.

Ah I missed to notice that "C" is used for copy. 

The reason I added the shortcut for clearing the prompt is to make the console view in sync with the most of the terminals(bash etc.) as they provide "Ctrl+C" to clear the prompt.