Closed
Bug 488410
Opened 15 years ago
Closed 15 years ago
autocomplete popup should display more than 7 entries, and should be pannable
Categories
(Firefox for Android Graveyard :: General, defect)
Firefox for Android Graveyard
General
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: Gavin, Assigned: Gavin)
Details
Attachments
(1 file, 1 obsolete file)
5.76 KB,
patch
|
mfinkle
:
review+
|
Details | Diff | Splinter Review |
No description provided.
Assignee | ||
Comment 1•15 years ago
|
||
I set a limit of 20 entries, but that is arbitrary. Currently doesn't work so great because mousing up after a pan still registers as a click, will need to sort things out with InputManager to fix that.
Assignee: nobody → gavin.sharp
Status: NEW → ASSIGNED
Comment 2•15 years ago
|
||
Comment on attachment 372762 [details] [diff] [review] patch Is there a downside to using richlistbox instead of scrollbox? We were hoping to create a "master" set of CSS styles for richlistboxes/items. Of course CSS is flexible enough to let us apply them here too.
Assignee | ||
Comment 3•15 years ago
|
||
richlistbox is just a lot heavier implementation wise (and requires a bunch of unnecessary richlistitem children, IIRC). I think we can probably manage to make our richlistbox styles work with this specific case of labels in a scrollbox fairly easily.
Assignee | ||
Comment 4•15 years ago
|
||
Attachment #372762 -
Attachment is obsolete: true
Attachment #375227 -
Flags: review?(mark.finkle)
Updated•15 years ago
|
Attachment #375227 -
Flags: review?(mark.finkle) → review+
Assignee | ||
Comment 5•15 years ago
|
||
https://hg.mozilla.org/mobile-browser/rev/20755fda5c49
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•