Bug 82602

Summary: Web Inspector: "go to the previous panel" shortcut is painful to maintain
Product: WebKit Reporter: Pavel Feldman <pfeldman>
Component: Web Inspector (Deprecated)Assignee: Pavel Feldman <pfeldman>
Status: RESOLVED FIXED    
Severity: Normal CC: apavlov, bweinstein, joepeck, keishi, loislo, pfeldman, pmuellr, rik, timothy, yurys
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
Patch
none
Patch
none
Patch webkit-ews: commit-queue-

Pavel Feldman
Reported 2012-03-29 05:48:01 PDT
Present go back shortcut "Cmd - Left" gets painful to support since we have more and more free flow editing capabilities (where Cmd - Left is handled by the editor). I'm suggesting that we remap it to Cmd - Option - Left (Ctrl - Alt - Left).
Attachments
Patch (10.71 KB, patch)
2012-03-29 06:40 PDT, Pavel Feldman 		no flags
DetailsFormatted DiffDiff
Patch (10.71 KB, patch)
2012-03-29 07:50 PDT, Pavel Feldman 		no flags
DetailsFormatted DiffDiff
Patch (10.72 KB, patch)
2012-03-29 08:01 PDT, Pavel Feldman 		webkit-ews: commit-queue-
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
Pavel Feldman
Comment 1 2012-03-29 06:40:14 PDT
Created attachment 134567 [details] Patch
Vsevolod Vlasov
Comment 2 2012-03-29 07:49:25 PDT
Comment on attachment 134567 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=134567&action=review > Source/WebCore/ChangeLog:10 > + Remaping it to Cmd - Option - Left (Ctrl - Alt - Left). Your ChangeLog is inconsistent with the code.
Pavel Feldman
Comment 3 2012-03-29 07:50:01 PDT
Created attachment 134583 [details] Patch
Vsevolod Vlasov
Comment 4 2012-03-29 07:57:51 PDT
Comment on attachment 134583 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=134583&action=review > Source/WebCore/ChangeLog:10 > + Remaping it to Cmd - Option - Left (Ctrl - Alt - [ (]) ). This looks weird. On the last line: Left -> [
Pavel Feldman
Comment 5 2012-03-29 08:01:37 PDT
Created attachment 134588 [details] Patch
Early Warning System Bot
Comment 6 2012-03-29 08:43:32 PDT
Comment on attachment 134588 [details] Patch Attachment 134588 [details] did not pass qt-wk2-ews (qt): Output: http://queues.webkit.org/results/12193370
Pavel Feldman
Comment 7 2012-03-29 09:22:38 PDT
Committed r112539: <http://trac.webkit.org/changeset/112539>
Vsevolod Vlasov
Comment 8 2012-03-29 11:00:48 PDT
Comment on attachment 134588 [details] Patch Clearing r?
Note You need to log in before you can comment on or make changes to this bug.