On Wed, Feb 23, 2011 at 7:03 PM, Sergiu Dumitriu <[email protected]> wrote: > On 02/23/2011 06:48 PM, Ecaterina Moraru (Valica) wrote: >> Hi, >> >> I've made a proposal on how to integrate keyboard shortcuts into UI >> http://incubator.myxwiki.org/xwiki/bin/view/Improvements/KeyboardShortcuts > > Looks good, but the changes to the QuickLinks panel are bad, IMO. > > - The (edit this panel) link is useful for new admins, since it's not > that straight-forward how to edit the quick links panel, without reading > a bit of documentation, so I'm -1 for removing it. > > - The "jump to any page" text is useful when the shortcuts don't work, > since clicking it also displays the jump dialog. For example, the > shortcut doesn't work for me in Chromium/KDE/Linux, where Ctrl+G > displays the browser's quick search, and Meta+G is captured as a simple > G which opens the WYSIWYG editor. Does it work on Mac or Windows?
Yes, it works on Mac, with the apple key and/or command key (depends on the browser). In my opinion, if we were to keep this shortcut, it should be in the panel content, not injected in JavaScript. People are complaining they can't find out how to remove it. Also Meta+G is HIGHLY technical for most users. There should at least be a tooltip that explains what it means. Or better, remove the Meta+G shorcut info from the panel, and put it directly in the jump to page light box, with visual representation of the shortcut like in Caty's proposal. Generally I think the box that sums up all shortcuts is good. On a case by case basis there should be shortcut information also displayed in context of the feature (like copy page, rename page, jump to page, etc.). This helps people discover and learn about shortcuts. Jerome > > -- > Sergiu Dumitriu > http://purl.org/net/sergiu/ > _______________________________________________ > devs mailing list > [email protected] > http://lists.xwiki.org/mailman/listinfo/devs > _______________________________________________ devs mailing list [email protected] http://lists.xwiki.org/mailman/listinfo/devs
