Bug 27286

Summary:

Inspector: Keyboard Shortcuts to Switch Panels

Product:

WebKit

Reporter:

Joseph Pecoraro <joepeck>

Component:

Web Inspector (Deprecated)

Assignee:

Nobody <webkit-unassigned>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

abarth

Priority:

Version:

528+ (Nightly build)

Hardware:

OS:

OS X 10.5

Attachments:

Description	Flags
Keyboard Shortcuts for Switching Panels	timothy: review-
Keyboard Shortcuts for Switching Panels	kmccullough: review+

Joseph Pecoraro

Reported 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.

Attachments
Keyboard Shortcuts for Switching Panels (2.89 KB, patch) 2009-07-14 18:48 PDT, Joseph Pecoraro	timothy: review-	Details Formatted Diff Diff
Keyboard Shortcuts for Switching Panels (2.84 KB, patch) 2009-07-23 13:54 PDT, Joseph Pecoraro	kmccullough: review+	Details Formatted Diff Diff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.

Joseph Pecoraro

Comment 1 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.

Timothy Hatcher

Comment 2 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.

Joseph Pecoraro

Comment 3 2009-07-23 13:54:43 PDT

Created attachment 33368 [details] Keyboard Shortcuts for Switching Panels Fix based on review.

Joseph Pecoraro

Comment 4 2009-07-23 13:56:05 PDT

(In reply to comment #2) > Should have spaces around the "-"s. Done.

Adam Barth

Comment 5 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 http://trac.webkit.org/changeset/46397

Note You need to log in before you can comment on or make changes to this bug.