diff options
author | Alexander Schmitz <arschmitz@gmail.com> | 2015-08-24 09:04:59 -0400 |
---|---|---|
committer | Alexander Schmitz <arschmitz@gmail.com> | 2015-09-11 08:36:40 -0400 |
commit | 14b3b607e53aa7c6a186e23be554911d5a56d1f7 (patch) | |
tree | c745b426779aaf5cb045d06af0c30bfe7ed19bc4 /ui/widgets/autocomplete.js | |
parent | b2bf10e0ec3719c37c86aab5c09208a9a5e5adbd (diff) | |
download | jquery-ui-14b3b607e53aa7c6a186e23be554911d5a56d1f7.tar.gz jquery-ui-14b3b607e53aa7c6a186e23be554911d5a56d1f7.zip |
Autocomplete: Style updates
Ref #14246
Diffstat (limited to 'ui/widgets/autocomplete.js')
-rw-r--r-- | ui/widgets/autocomplete.js | 14 |
1 files changed, 14 insertions, 0 deletions
diff --git a/ui/widgets/autocomplete.js b/ui/widgets/autocomplete.js index 954258e23..1ebd6ed7a 100644 --- a/ui/widgets/autocomplete.js +++ b/ui/widgets/autocomplete.js @@ -65,6 +65,7 @@ $.widget( "ui.autocomplete", { pending: 0, _create: function() { + // Some browsers only repeat keydown events, not keypress events, // so we use the suppressKeyPress flag to determine if we've already // handled the keydown event. #7269 @@ -120,8 +121,10 @@ $.widget( "ui.autocomplete", { this._keyEvent( "next", event ); break; case keyCode.ENTER: + // when menu is open and has focus if ( this.menu.active ) { + // #6055 - Opera still allows the keypress to occur // which causes forms to submit suppressKeyPress = true; @@ -140,6 +143,7 @@ $.widget( "ui.autocomplete", { this._value( this.term ); } this.close( event ); + // Different browsers have different default behavior for escape // Single press can mean undo or clear // Double press in IE means clear the whole form @@ -148,6 +152,7 @@ $.widget( "ui.autocomplete", { break; default: suppressKeyPressRepeat = true; + // search timeout should be triggered before the input value is changed this._searchTimeout( event ); break; @@ -210,6 +215,7 @@ $.widget( "ui.autocomplete", { this.menu = $( "<ul>" ) .appendTo( this._appendTo() ) .menu( { + // disable ARIA support, the live region takes care of that role: null } ) @@ -219,6 +225,7 @@ $.widget( "ui.autocomplete", { this._addClass( this.menu.element, "ui-autocomplete", "ui-front" ); this._on( this.menu.element, { mousedown: function( event ) { + // prevent moving focus out of the text field event.preventDefault(); @@ -259,6 +266,7 @@ $.widget( "ui.autocomplete", { }, menufocus: function( event, ui ) { var label, item; + // support: Firefox // Prevent accidental activation of menu items in Firefox (#7024 #9118) if ( this.isNewMenu ) { @@ -276,6 +284,7 @@ $.widget( "ui.autocomplete", { item = ui.item.data( "ui-autocomplete-item" ); if ( false !== this._trigger( "focus", event, { item: item } ) ) { + // use value to match what will end up in the input, if it was a key event if ( event.originalEvent && /^key/.test( event.originalEvent.type ) ) { this._value( item.value ); @@ -297,6 +306,7 @@ $.widget( "ui.autocomplete", { if ( this.element[ 0 ] !== $.ui.safeActiveElement( this.document[ 0 ] ) ) { this.element.trigger( "focus" ); this.previous = previous; + // #6109 - IE triggers two focus events and the second // is asynchronous, so we need to reset the previous // term synchronously and asynchronously :-( @@ -309,6 +319,7 @@ $.widget( "ui.autocomplete", { if ( false !== this._trigger( "select", event, { item: item } ) ) { this._value( item.value ); } + // reset the term after the select event // this allows custom select handling to work properly this.term = this._value(); @@ -473,6 +484,7 @@ $.widget( "ui.autocomplete", { this._suggest( content ); this._trigger( "open" ); } else { + // use ._close() instead of .close() so we don't cancel future searches this._close(); } @@ -499,6 +511,7 @@ $.widget( "ui.autocomplete", { }, _normalize: function( items ) { + // assume all items have the right format when the first item is complete if ( items.length && items[ 0 ].label && items[ 0 ].value ) { return items; @@ -538,6 +551,7 @@ $.widget( "ui.autocomplete", { _resizeMenu: function() { var ul = this.menu.element; ul.outerWidth( Math.max( + // Firefox wraps long text (possibly a rounding bug) // so we add 1px to avoid the wrapping (#7513) ul.width( "" ).outerWidth() + 1, |