RESOLVED WONTFIX 110548
Web Inspector: Console view should allow to clear the prompt in a quicker way
https://bugs.webkit.org/show_bug.cgi?id=110548
Summary Web Inspector: Console view should allow to clear the prompt in a quicker way
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.