Allow toggling elements with space
authorMarius Hoch <hoo@online.de>
Sat, 7 Sep 2013 12:46:35 +0000 (14:46 +0200)
committerMarius Hoch <hoo@online.de>
Sat, 7 Sep 2013 12:46:35 +0000 (14:46 +0200)
This seems more convenient than having the space key
do nothing as it does right now.
Furthermore this is more consistent with the various
other collapsible things we have around than to just
scroll the page on a space keypress.

Bug: 53797
Change-Id: If9292ef6a8fd82b097c597dc166e38c8ca31f4ff

resources/jquery/jquery.makeCollapsible.js

index ac46092..27c4c6c 100644 (file)
                        if ( e.type === 'click' && options.linksPassthru && $.nodeName( e.target, 'a' ) ) {
                                // Don't fire if a link was clicked, if requested  (for premade togglers by default)
                                return;
-                       } else if ( e.type === 'keypress' && e.which !== 13 ) {
-                               // Only handle keypresses on the "Enter" key
+                       } else if ( e.type === 'keypress' && e.which !== 13 && e.which !== 32 ) {
+                               // Only handle keypresses on the "Enter" or "Space" keys
                                return;
                        } else {
                                e.preventDefault();