675638 - richlistbox rows attribute is simply ignored

Status: RESOLVED INVALID

(Core :: XUL, defect)

Description

[Mike Kaply [:mkaply]]

The documentation for richlistbox indicated a rows attribute:

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.

From:

https://developer.mozilla.org/en/XUL/richlistbox#a-rows

This atttribute simply doesn't work. IF you set rows on a richlistbox and add 20 items, the richlistbox will grow vertically.

Looking at the code:

http://mxr.mozilla.org/mozilla2.0/source/toolkit/content/widgets/richlistbox.xml

I see no code to even handle this attribute.

Comment 1

[Neil Deakin]

The documentation should just be updated. richlistboxes don't have fixed row heights so determining the height from the number of rows to display isn't really possible.

Comment 2

[Mike Kaply [:mkaply]]

(In reply to Neil Deakin from comment #1)
> The documentation should just be updated. richlistboxes don't have fixed row
> heights so determining the height from the number of rows to display isn't
> really possible.

As I read that, a light came on. Duh :)

I'll fix the docs.

Comment 3

[Mike Kaply [:mkaply]]

I've updated the docs to remove rows from richlistbox.

Comment 4

[Firefox Bug Husbandry Bot]

Moving to Core:XUL per https://bugzilla.mozilla.org/show_bug.cgi?id=1455336