Closed Bug 1472554 Opened 6 years ago Closed 6 years ago

[meta] Convert "listbox" instances to "richlistbox"

Categories

(Firefox :: General, task)

task
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: Paolo, Unassigned)

References

Details

(Keywords: meta)

We have only a few remaining instances of the "listbox" element in production code. The one in "sanitize.xul" provides checkboxes, but all other cases just look like plain lists, and the only feature that they are using that is specific to "listbox" is the "rows" attribute to set the element height. We can easily convert these to use "richlistbox", and we can just set a height on the element that isn't a multiple of the row height, like most lists in the user interface already do. This allows us to remove several listbox-related bindings, as well as hundreds of lines of layout code that only supports features like the "rows" attribute.
Blocks: 1472555
Depends on: 1472716
Depends on: 1472744
Depends on: 1472748
Depends on: 1472750
Assignee: paolo.mozmail → nobody
Status: ASSIGNED → NEW
Keywords: meta
Priority: P1 → --
Depends on: 1473521
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Type: enhancement → task
Summary: Convert "listbox" instances to "richlistbox" → [meta] Convert "listbox" instances to "richlistbox"
You need to log in before you can comment on or make changes to this bug.