r932 - trunk/ui
r932 - trunk/ui
Author: david.bolter
Date: Thu Nov 13 08:24:44 2008
New Revision: 932
Modified:
trunk/ui/ui.accordion.js
Log:
refactored keyboard handling to a switch statement for clarity.
Modified: trunk/ui/ui.accordion.js
==============================================================================
--- trunk/ui/ui.accordion.js (original)
+++ trunk/ui/ui.accordion.js Thu Nov 13 08:24:44 2008
@@ -106,18 +106,18 @@
var currentIndex = this.options.headers.index(e.target);
var toFocus = false;
- if (e.keyCode == keyCode.RIGHT || e.keyCode == keyCode.DOWN){
-
- toFocus = this.options.headers[(currentIndex + 1) % length];
-
- } else if (e.keyCode == keyCode.LEFT || e.keyCode == keyCode.UP) {
-
- toFocus = this.options.headers[(currentIndex - 1 + length) % length];
-
- } else if (e.keyCode == keyCode.SPACE || e.keyCode == keyCode.ENTER) {
-
- return clickHandler.call(this.element[0], { target: e.target });
-
+ switch(e.keyCode) {
+ case keyCode.RIGHT:
+ case keyCode.DOWN:
+ toFocus = this.options.headers[(currentIndex + 1) % length];
+ break;
+ case keyCode.LEFT:
+ case keyCode.UP:
+ toFocus = this.options.headers[(currentIndex - 1 + length) % length];
+ break;
+ case keyCode.SPACE:
+ case keyCode.ENTER:
+ return clickHandler.call(this.element[0], { target: e.target });
}
if (toFocus) {