Closed Bug 1914687 Opened 8 months ago Closed 8 months ago

"scrollbutton-up" of long bookmarks menupopup no longer working

Categories

(Firefox :: Menus, defect)

Firefox 131
Desktop
All
defect

Tracking

()

VERIFIED FIXED
131 Branch
Tracking Status
firefox-esr115 --- unaffected
firefox-esr128 --- unaffected
firefox129 --- unaffected
firefox130 --- unaffected
firefox131 --- verified

People

(Reporter: alice0775, Assigned: emilio)

References

(Regression)

Details

(Keywords: nightly-community, regression)

Attachments

(1 file)

Steps to reproduce:

  1. Bookmark a lot of Bookmarks so that Bookmarks menupopup can be scrolled.
  2. Expand the Bookmarks menu
  3. Scroll the menupopup to bottom in any way.
  4. Attempt to scroll the menupopup to top by using ^ scrollbutton-up button

Actual results:
The "scrollbutton-up" does not work.
No mouse hoover effect.
Not scroll when mouse over it

Expected results:
Change style when mouse hover it
Should start scrolling when mouse over it

Regression window:
https://hg.mozilla.org/integration/autoland/pushloghtml?fromchange=773aed942ff98525ee9bc2c03fd96dbd6c0eee96&tochange=9d4782d16aacb9b51e440042a677f6c5c68a0ca9

Summary: ` ^ ` "scrollbutton-up" of long bookmarks menupopup no longer working → "scrollbutton-up" of long bookmarks menupopup no longer working

:emilio, since you are the author of the regressor, bug 1913322, could you take a look? Also, could you set the severity field?

For more information, please visit BugBot documentation.

Flags: needinfo?(emilio)

Reproduced on Windows11, ubuntu22.04 and Linux Mint21 cinnamon.

OS: Windows 11 → All

In bug 1913322 I tweaked the approach we use to maintain layout
stability when showing and hiding these so that the scrollbox overlaps
them (before we used to have an extra clip).

The DOM order is scrollbutton-up, scrollbox, scrollbutton-down, but we
need to make sure that the scrollbutton-up paint on top of the
scrollbox, otherwise it won't be hittable.

Assignee: nobody → emilio
Status: NEW → ASSIGNED
Flags: needinfo?(emilio)
Pushed by ealvarez@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/aacc1a11d786 Make sure arrowscrollbox buttons draw on top of scroller. r=dao,desktop-theme-reviewers
Status: ASSIGNED → RESOLVED
Closed: 8 months ago
Resolution: --- → FIXED
Target Milestone: --- → 131 Branch

I was able to reproduce the issue on an affected Firefox Nightly build from 2024-08-23, on Windows 11, while following the steps from Comment 0.
Verified as fixed on Firefox 131.0b4, using Windows 11, Ubuntu 22.04 and macOS 14.6. The "scrollbutton-up" button is now working.

Status: RESOLVED → VERIFIED
Flags: qe-verify+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: