"?" (show shortcuts page) is not mentioned on shortcuts page.
Created attachment 193509 [details] Patch
Comment on attachment 193509 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=193509&action=review > Source/WebCore/inspector/front-end/inspector.js:704 > + section.addKey(shortcut.makeDescriptor("?"), WebInspector.UIString("Show this page")); Show keyboard shortcuts help
Comment on attachment 193509 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=193509&action=review >> Source/WebCore/inspector/front-end/inspector.js:704 >> + section.addKey(shortcut.makeDescriptor("?"), WebInspector.UIString("Show this page")); > > Show keyboard shortcuts help Please note that F1 also shows this help page.
Created attachment 193558 [details] Patch
Created attachment 193561 [details] Patch
Comment on attachment 193509 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=193509&action=review >>> Source/WebCore/inspector/front-end/inspector.js:704 >>> + section.addKey(shortcut.makeDescriptor("?"), WebInspector.UIString("Show this page")); >> >> Show keyboard shortcuts help > > Please note that F1 also shows this help page. OK Done.
Comment on attachment 193561 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=193561&action=review > Source/WebCore/inspector/front-end/inspector.js:-678 > - var keys = [ 'var' should be here, not below.
Created attachment 193563 [details] Patch
Comment on attachment 193561 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=193561&action=review >> Source/WebCore/inspector/front-end/inspector.js:-678 >> - var keys = [ > > 'var' should be here, not below. surely. it is above =)
Comment on attachment 193563 [details] Patch Clearing flags on attachment: 193563 Committed r146066: <http://trac.webkit.org/changeset/146066>
All reviewed patches have been landed. Closing bug.