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

Status: RESOLVED INVALID

(Toolkit :: UI Widgets, defect)

Description

[(not reading, please use [e-mail] instead)]

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.

Comment 1

[Dão Gottwald [:dao]]

Elements in a richlistbox don't necessarily have the same height. Could this bug be invalid?

Comment 2

[Neil Deakin]

Yes, this doesn't seem like it would make sense.

Comment 3

[Ed Lee :Mardak]

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.

Comment 4

[Dão Gottwald [:dao]]

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.