Closed Bug 1533693 Opened 2 years ago Closed 2 years ago

If the menupopup has been scrolled, the subfolder would not open when dragging over the subfolder menu.

Categories

(Core :: XUL, defect, P1)

67 Branch
x86_64
Windows 10
defect

Tracking

()

VERIFIED FIXED
mozilla67
Tracking Status
firefox-esr60 --- unaffected
firefox65 --- unaffected
firefox66 --- unaffected
firefox67 + verified

People

(Reporter: alice0775, Assigned: enndeakin)

References

Details

(Keywords: nightly-community, regression)

Attachments

(1 file)

[Tracking Requested - why for this release]: break bookmarks menupopup

Reproducible: always

Steps to reproduce:
0. Prepare long bookmark menu and some folders in the menu so that the menu can be scrolled
i,e
Bookmarks(or bookmarl folder)
bookmark
bookmark
bookmark
bookmark
bookmark
bookmark
bookmark folder
bookmark folder
bookmark
bookmark
bookmark
bookmark
...
bookmark
bookmark

  1. Click bookmarks menu or bookmarks folder in bookmark toolbar
  2. Scroll the menupopup a little (if not scrolled, this problem does not happen)
  3. Dragging a bookmark over a bookmark folder submenu

Actual Results:
The bookmark folder submenu would not open

Expected Results:
The bookmark folder submenu should open automatically

Regression window:https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=fad9ff877c1cbd2b06c03263ba9fc91d2f1fc405&tochange=57624be2e55cefabe21feb5e0169f33b283f9ddd

Suspect: aa8dea3c0f43 Neil Deakin — Bug 1519953, replace calls to retrieve boxobject position and size with getBoundingClientRect, r=gijs

Neil, Your bunch of patch seems to cause the regression. Can you please look into this?

Flags: needinfo?(enndeakin)
See Also: → 1533720
Flags: needinfo?(enndeakin)
Priority: -- → P1
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla67
Assignee: nobody → enndeakin
Flags: qe-verify+

I’ve reproduced the issue with Fx 67.0a1 (2019-03-08) on Windows 10 x64.
This issue is verified fixed on Fx 67.0b11 on Windows 10 x64, Ubuntu 18.04 x64 and macOS 10.13.

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