diff options
| author | Scott González <scott.gonzalez@gmail.com> | 2012-02-09 14:00:21 -0500 |
|---|---|---|
| committer | Scott González <scott.gonzalez@gmail.com> | 2012-02-09 14:00:40 -0500 |
| commit | 470030249c519eb59d30d81e6fd6c989490fd5b8 (patch) | |
| tree | cb289bf2affcb615e5ac3ac7d9ce59c7f7caa7a1 /ui/jquery.ui.slider.js | |
| parent | dc12b4d3109d2b5f611ce2bd8d9caae484cc772b (diff) | |
| download | jquery-ui-470030249c519eb59d30d81e6fd6c989490fd5b8.tar.gz jquery-ui-470030249c519eb59d30d81e6fd6c989490fd5b8.zip | |
Slider: Prevent the default action on keydown if we're listening for the pressed key. Fixes #8109 - Slider: Key handling should prevent scrolling.
(cherry picked from commit 0434bce03d1368f5ace043fa50e9d4f95677c09e)
Diffstat (limited to 'ui/jquery.ui.slider.js')
| -rw-r--r-- | ui/jquery.ui.slider.js | 8 |
1 files changed, 2 insertions, 6 deletions
diff --git a/ui/jquery.ui.slider.js b/ui/jquery.ui.slider.js index 457362577..964597ae9 100644 --- a/ui/jquery.ui.slider.js +++ b/ui/jquery.ui.slider.js @@ -115,8 +115,7 @@ $.widget( "ui.slider", $.ui.mouse, { this.handles .keydown(function( event ) { - var ret = true, - index = $( this ).data( "index.ui-slider-handle" ), + var index = $( this ).data( "index.ui-slider-handle" ), allowed, curVal, newVal, @@ -135,7 +134,7 @@ $.widget( "ui.slider", $.ui.mouse, { case $.ui.keyCode.RIGHT: case $.ui.keyCode.DOWN: case $.ui.keyCode.LEFT: - ret = false; + event.preventDefault(); if ( !self._keySliding ) { self._keySliding = true; $( this ).addClass( "ui-state-active" ); @@ -184,9 +183,6 @@ $.widget( "ui.slider", $.ui.mouse, { } self._slide( event, index, newVal ); - - return ret; - }) .keyup(function( event ) { var index = $( this ).data( "index.ui-slider-handle" ); |