Closed Bug 1391625 Opened 4 years ago Closed 4 years ago
Sidebar items spacing should change according to "Density" option
User Agent: Mozilla/5.0 (X11; Linux x86_64; rv:57.0) Gecko/20100101 Firefox/57.0 Build ID: 20170818100226 Steps to reproduce: Open sidebar for bookmarks or history. Actual results: The spacing between items, as of 8/17, got much larger. My bookmarks now run off the bottom of the screen, and it is hard to scroll down. I had them carefully arranged so that they would all fit. Expected results: The spacing should not have changed. There are addons to fix this, but they are no longer compatible with Nightly, and userChrome seems not to be supported anymore, although I have not tried just creating it. I should not have to do this, though. This extra spacing is harmful, not helpful.
Unfortunately, looks like it's intentional per bug #1391625, maybe the spacing between Bookmark item Icon and Label should change according to chosen density option, so in compact mode the spacing will be much smaller. @ Stephen Horlander [:shorlander] - What do you think about this?
Also patch from bug #1388699 will add more space again, so adding that bug too. And it could be caused by patch from bug #1388700, not from patch from bug #1391331.
Has Regression Range: --- → yes
Has STR: --- → yes
Summary: sidebar spacing too large → Sidebar spacing too large
Whiteboard: [photon-visual] [triage]
Changing title from "Sidebar spacing too large" to "Sidebar/Toolbar spacing should change according to "Density" option" to catch all space, as it also should change, same as button sizes and spacing between tab bar, location bar and bookmarks toolbar, because in other way it will odd, that e.g. everything is compacted in "Compact" density mode, except only bookmark button items and their spaces are not.
4 years ago
See Also: → 1391593
4 years ago
Summary: Sidebar/Toolbar items spacing should change according to "Density" option → Sidebar/Toolbar/Menu items spacing should change according to "Density" option
4 years ago
Depends on: 1397086
No longer depends on: 1397086
Duplicate of this bug: 1397086
4 years ago
Depends on: 1374815
what's the status and priority of this?
This bug seems to have originally been about the bookmarks sidebar and we should keep it at that. The bookmarks toolbar already adjusts to UI density. Making the sidebar adjust to UI density is not in scope for Photon MVP, though I personally think it's a good idea and we should pursue it eventually, as we've gotten a lot of positive feedback on UI density as a general concept. But yeah, unfortunately not in scope for 57/Photon.
Summary: Sidebar/Toolbar/Menu items spacing should change according to "Density" option → Sidebar items spacing should change according to "Density" option
Ah, I'm already working on this in bug 1389094. I'll dupe this one.
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → DUPLICATE
Duplicate of bug: 1389094
This bug was not resolved. The original problem was that the bookmark menu spacing was too large. The problem arose because the "zoom page" add-on stopped working, and I had to use layout.css.devPixelsPerPx in order to make pages readable. This applied to the sidebar as well as the page content, and the bookmarks sidebar became un-usable. The problem has now been solved by the "zoom" add-on. It is a little annoying that this bug was just subsumed into other bugs that had nothing do do with the original problem. Let's hope this add-on continues to work.
Thanks for clarifying Jonathan. If the add-on stops working we can reopen the bug. It also looks like this wasn't fixed in the maybe-not-duplicate bug in 57.
You need to log in before you can comment on or make changes to this bug.