Bug 110548

Summary: Web Inspector: Console view should allow to clear the prompt in a quicker way
Product: WebKit Reporter: Vivek Galatage <vivekg>
Component: Web Inspector (Deprecated)Assignee: Vivek Galatage <vivekg>
Status: RESOLVED WONTFIX    
Severity: Normal CC: apavlov, keishi, loislo, pfeldman, pmuellr, vsevik, web-inspector-bugs, yurys
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
Patch vsevik: review-

Vivek Galatage
Reported 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.
Attachments
Patch (2.42 KB, patch)
2013-02-21 20:43 PST, Vivek Galatage
vsevik: review-
Vivek Galatage
Comment 1 2013-02-21 20:43:50 PST
Eugene Klyuchnikov
Comment 2 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.
Vsevolod Vlasov
Comment 3 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.
Vivek Galatage
Comment 4 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.
Note You need to log in before you can comment on or make changes to this bug.