Bug 27286

Summary: Inspector: Keyboard Shortcuts to Switch Panels
Product: WebKit Reporter: Joseph Pecoraro <joepeck>
Component: Web Inspector (Deprecated)Assignee: Nobody <webkit-unassigned>
Severity: Normal CC: abarth
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: PC   
OS: OS X 10.5   
Description Flags
Keyboard Shortcuts for Switching Panels
timothy: review-
Keyboard Shortcuts for Switching Panels kmccullough: review+

Description Joseph Pecoraro 2009-07-14 18:29:53 PDT
It would be nice to flip between Web Inspector Panels with keyboard shortcuts.  They are much like tabs in the browser.
Comment 1 Joseph Pecoraro 2009-07-14 18:48:01 PDT
Created attachment 32758 [details]
Keyboard Shortcuts for Switching Panels

Some of the more obvious keyboard shortcuts are taken by Safari.  Like ⌘1, ⌘2, etc. are hot links to bookmarks and Safari grabs them.  It also seems to grab ⇧⌘[ and ⇧⌘] probably because it thinks an undocked Inspector is a window.

So I resorted to ⌘] and ⌘[ for Macs.
I don't actually have a Windows Safari setup but I did ^[ and ^].

Let me know if you can think of other keyboard shortcuts for any features or better ones for panel manipulation.
Comment 2 Timothy Hatcher 2009-07-23 11:40:21 PDT
Comment on attachment 32758 [details]
Keyboard Shortcuts for Switching Panels

> +                    index = (index === 0) ? this.panelOrder.length-1 : index-1;

Should have spaces around the "-"s.
Comment 3 Joseph Pecoraro 2009-07-23 13:54:43 PDT
Created attachment 33368 [details]
Keyboard Shortcuts for Switching Panels

Fix based on review.
Comment 4 Joseph Pecoraro 2009-07-23 13:56:05 PDT
(In reply to comment #2)
> Should have spaces around the "-"s.

Comment 5 Adam Barth 2009-07-25 14:35:01 PDT
Committing to http://svn.webkit.org/repository/webkit/trunk ...
	M	WebCore/ChangeLog
	M	WebCore/inspector/front-end/inspector.js
Committed r46397
	M	WebCore/ChangeLog
	M	WebCore/inspector/front-end/inspector.js
r46397 = 1c431f0eb9ac3e3fdcb5cae52a97bf531525a6b7 (trunk)
No changes between current HEAD and refs/remotes/trunk
Resetting to the latest refs/remotes/trunk