Open Bug 1707717 Opened 5 months ago Updated 2 months ago

Bookmarks panel goes beyond screen when fully expanded

Categories

(Firefox :: Foxfooding, defect, P2)

Firefox 89
Desktop
Unspecified
defect

Tracking

()

People

(Reporter: pdehaan, Unassigned)

References

(Blocks 1 open bug)

Details

(Whiteboard: [foxfooding][community] [proton-foxfooding] [proton-doorhangers] [priority:2b])

Attachments

(2 files)

This comment is copy/pasted from a community issue filed during the Foxfooding program.


Firefox Version: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:89.0) Gecko/20100101 Firefox/89.0
Window Size (inner width and height): 1536x711

Steps to Reproduce

  1. Click on Bookmark this Page
  2. Expand Show All Bookmarks folder
  3. Expand Tags

Expected Behavior

Fit to screen

Actual Behavior

The Save and Remove bookmark buttons go beyond screen.

Priority: -- → P2
Whiteboard: [foxfooding][community] [proton-foxfooding] → [foxfooding][community] [proton-foxfooding] [proton-doorhangers] [priority:2b]
Summary: Bookmarks go beyond screen when fully expanded → Bookmarks panel goes beyond screen when fully expanded
Duplicate of this bug: 1707693
Assignee: nobody → bigiri

Could you please add a screenshot of this issue?

Flags: needinfo?(pdehaan)
Status: NEW → ASSIGNED

In some cases I can see scrollbars on macOS when expanding both the Bookmarks and Tags sections. Other times, if the window if full width/height [not maximized, just full available screen size], the pop up panel will not show any scrollbars. So I think this is only showing scrollbars on small resolution screens or when things won't absolutely fit.

Flags: needinfo?(pdehaan)

Screenshot showing the window at full width/height on my macOS laptop and no scrollbars when opening both sections in the Edit bookmark panel.

This is still unclear. I think we need UX feedback as there are many potential solutions for this, from shrinking the height of certain elements, to rearranging the layout on smaller resolutions, to keeping the Remove and Save buttons outside of the scrolling div. Blake Winton suggested putting this in the “Blocked Issues” sprint. I'm not sure how to do that unfortunately.

Assignee: bigiri → nobody
Status: ASSIGNED → NEW
Flags: needinfo?(pdehaan)

This is now in the "Blocked Issues" sprint.

Flags: needinfo?(pdehaan)
See Also: → 1492235
See Also: 1492235
You need to log in before you can comment on or make changes to this bug.