From: Scott González Date: Tue, 16 Feb 2016 15:22:45 +0000 (-0500) Subject: Autocomplete: Work around `isContentEditable` bug in Chrome X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=eb6d623589b50e9a1a39492d41914de01945be0c;p=jquery-ui.git Autocomplete: Work around `isContentEditable` bug in Chrome Fixes #14917 Closes gh-1673 (cherry picked from commit cbceca7091b17e05c5a9ba887ef54761568bb70b) --- diff --git a/ui/autocomplete.js b/ui/autocomplete.js index 0741dadb1..fe14726be 100644 --- a/ui/autocomplete.js +++ b/ui/autocomplete.js @@ -71,7 +71,7 @@ $.widget( "ui.autocomplete", { // Inputs are always single-line, even if inside a contentEditable element // IE also treats inputs as contentEditable // All other element types are determined by whether or not they're contentEditable - this.isMultiLine = isTextarea || !isInput && this.element.prop( "isContentEditable" ); + this.isMultiLine = isTextarea || !isInput && this._isContentEditable( this.element ); this.valueMethod = this.element[ isTextarea || isInput ? "val" : "text" ]; this.isNewMenu = true; @@ -585,6 +585,24 @@ $.widget( "ui.autocomplete", { // prevents moving cursor to beginning/end of the text field in some browsers event.preventDefault(); } + }, + + // Support: Chrome <=50 + // We should be able to just use this.element.prop( "isContentEditable" ) + // but hidden elements always report false in Chrome. + // https://code.google.com/p/chromium/issues/detail?id=313082 + _isContentEditable: function( element ) { + if ( !element.length ) { + return false; + } + + var editable = element.prop( "contentEditable" ); + + if ( editable === "inherit" ) { + return this._isContentEditable( element.parent() ); + } + + return editable === "true"; } });