Skip to content

Commit 0e1bfaf

Browse files
committed
Merge remote branch 'treyhunner/topic/arrow-keys'
2 parents 309ca08 + bdfc6d5 commit 0e1bfaf

File tree

1 file changed

+26
-0
lines changed

1 file changed

+26
-0
lines changed

ui/jquery.ui.autocomplete.js

Lines changed: 26 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -62,24 +62,29 @@ $.widget( "ui.autocomplete", {
6262
})
6363
.bind( "keydown.autocomplete", function( event ) {
6464
if ( self.options.disabled || self.element.attr( "readonly" ) ) {
65+
suppressKeyPress = true;
6566
return;
6667
}
6768

6869
suppressKeyPress = false;
6970
var keyCode = $.ui.keyCode;
7071
switch( event.keyCode ) {
7172
case keyCode.PAGE_UP:
73+
suppressKeyPress = true;
7274
self._move( "previousPage", event );
7375
break;
7476
case keyCode.PAGE_DOWN:
77+
suppressKeyPress = true;
7578
self._move( "nextPage", event );
7679
break;
7780
case keyCode.UP:
81+
suppressKeyPress = true;
7882
self._move( "previous", event );
7983
// prevent moving cursor to beginning of text field in some browsers
8084
event.preventDefault();
8185
break;
8286
case keyCode.DOWN:
87+
suppressKeyPress = true;
8388
self._move( "next", event );
8489
// prevent moving cursor to end of text field in some browsers
8590
event.preventDefault();
@@ -121,7 +126,28 @@ $.widget( "ui.autocomplete", {
121126
if ( suppressKeyPress ) {
122127
suppressKeyPress = false;
123128
event.preventDefault();
129+
return;
124130
}
131+
132+
var keyCode = $.ui.keyCode;
133+
switch( event.keyCode ) {
134+
case keyCode.PAGE_UP:
135+
self._move( "previousPage", event );
136+
break;
137+
case keyCode.PAGE_DOWN:
138+
self._move( "nextPage", event );
139+
break;
140+
case keyCode.UP:
141+
self._move( "previous", event );
142+
// prevent moving cursor to beginning of text field in some browsers
143+
event.preventDefault();
144+
break;
145+
case keyCode.DOWN:
146+
self._move( "next", event );
147+
// prevent moving cursor to end of text field in some browsers
148+
event.preventDefault();
149+
break;
150+
}
125151
})
126152
.bind( "focus.autocomplete", function() {
127153
if ( self.options.disabled ) {

0 commit comments

Comments
 (0)