support "rows" attribute on richlistbox (like we do on listbox)

RESOLVED INVALID

Status

()

defect
RESOLVED INVALID
12 years ago
11 years ago

People

(Reporter: moco, Unassigned)

Tracking

(Blocks 1 bug)

Trunk
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

support "rows" attribute on richlistbox (like we do on listbox)

from http://developer.mozilla.org/en/docs/XUL:listbox

 rows
    Type: integer 
    The number of rows to display in the element. If the element contains more than this number of rows, a scrollbar will appear which the user can use to scroll to the other rows. To get the actual number of rows in the element, use the getRowCount method. 

I'd like this so that for bug #399664 I could set the rows on richlistbox (that holds all the richlistitems for autocomplete results) without needing to calculate the pixel height.
Elements in a richlistbox don't necessarily have the same height. Could this bug be invalid?
Yes, this doesn't seem like it would make sense.
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → INVALID
Couldn't an implementation of richlistbox with rows have it automatically size the richlistbox to fit "rows" number of elements before showing the scrollbar. Anything afterwards will be scrolled in even if they might be partially cropped because they're different sizes.
I guess that could be done in the constructor and when elements are added or removed. Doesn't seem very appealing though, because:

1) we can do that in an optimized and more efficient manner in the
   autocomplete binding,
2) there aren't other use cases or potential consumers (are there?),
3) there are edge cases that still wouldn't be supported well, e.g.
   the add-ons manager where the selected item gets bigger,
4) you really want a "richlisttree" anyway (bug 405913).

Following this, it's probably more WONTFIX than INVALID.
You need to log in before you can comment on or make changes to this bug.