Closed
Bug 1391625
Opened 7 years ago
Closed 7 years ago
Sidebar items spacing should change according to "Density" option
Categories
(Firefox :: Bookmarks & History, defect, P3)
Tracking
()
RESOLVED
DUPLICATE
of bug 1389094
Tracking | Status | |
---|---|---|
firefox-esr52 | --- | unaffected |
firefox55 | --- | unaffected |
firefox56 | --- | unaffected |
firefox57 | --- | wontfix |
People
(Reporter: jonathanbaron7, Unassigned)
References
Details
(Keywords: regression, ux-consistency, ux-minimalism)
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.
Updated•7 years ago
|
Component: Untriaged → Bookmarks & History
Comment 1•7 years ago
|
||
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?
Blocks: 1391331
Flags: needinfo?(shorlander)
Comment 2•7 years ago
|
||
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.
Blocks: 1388699
Has Regression Range: --- → yes
Has STR: --- → yes
status-firefox55:
--- → unaffected
status-firefox56:
--- → unaffected
status-firefox57:
--- → affected
status-firefox-esr52:
--- → unaffected
Keywords: regression,
ux-minimalism
Summary: sidebar spacing too large → Sidebar spacing too large
Whiteboard: [photon-visual] [triage]
Comment 3•7 years ago
|
||
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.
Blocks: 1388700
Severity: normal → major
Keywords: ux-consistency
OS: Unspecified → All
Hardware: Unspecified → All
Summary: Sidebar spacing too large → Sidebar/Toolbar items spacing should change according to "Density" option
Updated•7 years ago
|
Updated•7 years ago
|
Whiteboard: [photon-visual] [triage]
Updated•7 years ago
|
Summary: Sidebar/Toolbar items spacing should change according to "Density" option → Sidebar/Toolbar/Menu items spacing should change according to "Density" option
Comment 6•7 years ago
|
||
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.
Flags: needinfo?(jhofmann)
Summary: Sidebar/Toolbar/Menu items spacing should change according to "Density" option → Sidebar items spacing should change according to "Density" option
Updated•7 years ago
|
Blocks: photon-touch
Updated•7 years ago
|
Severity: major → normal
Priority: -- → P3
Comment 7•7 years ago
|
||
Ah, I'm already working on this in bug 1389094. I'll dupe this one.
Status: NEW → RESOLVED
Closed: 7 years ago
Flags: needinfo?(shorlander)
Resolution: --- → DUPLICATE
Updated•7 years ago
|
No longer blocks: photon-touch
Reporter | ||
Comment 8•7 years ago
|
||
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.
Comment 9•7 years ago
|
||
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.
Description
•