Closed
Bug 401948
Opened 17 years ago
Closed 6 years ago
autocomplete popup code, use the popup's open state directly
Categories
(Toolkit :: Autocomplete, defect)
Toolkit
Autocomplete
Tracking
()
RESOLVED
INACTIVE
People
(Reporter: moco, Unassigned)
References
Details
autocomplete popup code, use the popup's open state directly in bug #399664 comment #22, neil writes: >+ <handlers> >+ <handler event="popupshowing"><![CDATA[ >+ // If normalMaxRows wasn't already set by the input, then set it here >+ // so that we restore the correct number when the popup is hidden. >+ if (this._normalMaxRows < 0) >+ this._normalMaxRows = this.mInput.maxRows; >+ >+ this.mPopupOpen = true; >+ ]]></handler> "The popup won't necessarily be opened, for instance if another listener called preventDefault. You should just get the popup's open state directly when needed via its 'state' property." note, both the existing autocomplete tree based popup code and the upcoming richlistbox version has this problem
Reporter | ||
Updated•17 years ago
|
Component: General → Autocomplete
Product: Firefox → Toolkit
QA Contact: general → autocomplete
Reporter | ||
Comment 1•17 years ago
|
||
from gavin:
>+ <property name="overrideValue" readonly="true"
>+ onget="return null;"/>
>+
>+ <property name="popupOpen" readonly="true"
>+ onget="return this.mPopupOpen;"/>
make this use this.mPopup.popupState as Neil mentions in his comment.
Comment 2•6 years ago
|
||
Per policy at https://wiki.mozilla.org/Bug_Triage/Projects/Bug_Handling/Bug_Husbandry#Inactive_Bugs. If this bug is not an enhancement request or a bug not present in a supported release of Firefox, then it may be reopened.
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → INACTIVE
You need to log in
before you can comment on or make changes to this bug.
Description
•