Skip to content

Commit 0bb89d8

Browse files
committed
Merge pull request usablica#204 from ximi/optional-keyboard-navigation
Make keyboard navigation a configurable option
2 parents 5e84e4e + c51be52 commit 0bb89d8

File tree

1 file changed

+11
-5
lines changed

1 file changed

+11
-5
lines changed

intro.js

Lines changed: 11 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,9 @@
4747
/* Close introduction when clicking on overlay layer? */
4848
exitOnOverlayClick: true,
4949
/* Show step numbers in introduction? */
50-
showStepNumbers: true
50+
showStepNumbers: true,
51+
/* Let user use keyboard to navigate the tour? */
52+
keyboardNavigation: true
5153
};
5254
}
5355

@@ -131,7 +133,7 @@
131133
//right arrow or enter
132134
_nextStep.call(self);
133135
//prevent default behaviour on hitting Enter, to prevent steps being skipped in some browsers
134-
if(e.preventDefault) {
136+
if(e.preventDefault) {
135137
e.preventDefault();
136138
} else {
137139
e.returnValue = false;
@@ -144,11 +146,15 @@
144146
};
145147

146148
if (window.addEventListener) {
147-
window.addEventListener('keydown', self._onKeyDown, true);
149+
if(this._options.keyboardNavigation) {
150+
window.addEventListener('keydown', self._onKeyDown, true);
151+
}
148152
//for window resize
149153
window.addEventListener("resize", self._onResize, true);
150154
} else if (document.attachEvent) { //IE
151-
document.attachEvent('onkeydown', self._onKeyDown);
155+
if(this._options.keyboardNavigation) {
156+
document.attachEvent('onkeydown', self._onKeyDown);
157+
}
152158
//for window resize
153159
document.attachEvent("onresize", self._onResize);
154160
}
@@ -601,7 +607,7 @@
601607
rect.top >= 0 &&
602608
rect.left >= 0 &&
603609
(rect.bottom+80) <= window.innerHeight && // add 80 to get the text right
604-
rect.right <= window.innerWidth
610+
rect.right <= window.innerWidth
605611
);
606612
}
607613

0 commit comments

Comments
 (0)