Skip to content
Permalink
Browse files

slider: only return false on handle keydown if one of arrow keys or p…

…gup/down are pressed (otherwise tab is broken)
  • Loading branch information...
rdworth committed Feb 23, 2009
1 parent 12d2b5e commit 639da64b3482fd717f76fb7d36e8d43d3b584945
Showing with 4 additions and 1 deletion.
  1. +4 −1 ui/ui.slider.js
@@ -88,6 +88,8 @@ $.widget("ui.slider", $.extend({}, $.ui.mouse, {

this.handles.keydown(function(event) {

var ret = true;

var index = $(this).data("index.ui-slider-handle");

if (self.options.disabled)
@@ -100,6 +102,7 @@ $.widget("ui.slider", $.extend({}, $.ui.mouse, {
case $.ui.keyCode.RIGHT:
case $.ui.keyCode.DOWN:
case $.ui.keyCode.LEFT:
ret = false;
if (!self._keySliding) {
self._keySliding = true;
$(this).addClass("ui-state-active");
@@ -136,7 +139,7 @@ $.widget("ui.slider", $.extend({}, $.ui.mouse, {

self._slide(event, index, newVal);

return false;
return ret;

}).keyup(function(event) {

0 comments on commit 639da64

Please sign in to comment.
You can’t perform that action at this time.