345399 - arrowscrollbox - update scroll buttons disabled state when the scrollbox width is changed

Status: RESOLVED FIXED

(Toolkit :: UI Widgets, defect)

Description

[Asaf Romano (gone)]

In the arrowscrollbox binding, we're disabling a scroll button when there's no where to scroll in the direction it points to. We're updating its disabled state once the scrollbox overflows (at the same time we show the scroll buttons) and when the scrollbox is scrolled.

However, we don't update its disabled state when the width of the boxObject of the scrollbox is changed, see bug 343585 for example.

See bug 343585 comment 2 for a proposed solution.

Comment 1

[Neil Deakin]

(In reply to comment #0)
> See bug 343585 comment 2 for a proposed solution.

Well we can't add a 'box has changed size event' since that would require firing many events at a time, but it could be done by supporting a 'resize' event on the popup. Is that sufficient for this case?

Comment 2

[Dão Gottwald [:dao]]

Attachment: Bug 345399 - Update scroll buttons' disabled state when the scrollbox width changes. r=emilio!,dwalker!

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/5a856bace4f9
Update scroll buttons' disabled state when the scrollbox width changes. r=emilio,dwalker,tabbrowser-reviewers

Comment 4

[Atila Butkovits]

https://hg.mozilla.org/mozilla-central/rev/5a856bace4f9