GNOME Bugzilla – Bug 718611
Allow thinner minimum sidebar width
Last modified: 2015-05-04 08:25:34 UTC
---- Reported by shotwell-maint@gnome.bugs 2012-05-06 12:45:00 -0700 ---- Original Redmine bug id: 5214 Original URL: http://redmine.yorba.org/issues/5214 Searchable id: yorba-bug-5214 Original author: Daniel Fore Original description: The current minimum sidebar width is quite large when working on a smaller display. It would be great if I could set it to something thinner, maybe more like 160px? ---- Additional Comments From shotwell-maint@gnome.bugs 2013-09-17 12:05:00 -0700 ---- ### History #### #1 Updated by Adam Dingle over 1 year ago * **Priority** changed from _Normal_ to _High_ * **Target version** set to _0.13_ Sounds reasonable. #### #2 Updated by Clinton Rogers about 1 year ago * **File** 5214.patch added * **Status** changed from _Open_ to _Review_ * **% Done** changed from _0_ to _80_ #### #3 Updated by G. W. about 1 year ago I agree with this, but I want the ability to completely hide the sidepane even more than this. the same way I can press F9 key to make the sidepane disappear/appear in Nautilus. This would be valuable feature in Shotwell as well. #### #4 Updated by Adam Dingle about 1 year ago * **Status** changed from _Review_ to _Open_ * **Target version** deleted (<strike>_0.13_</strike>) * **% Done** changed from _80_ to _0_ I'll review since I know Lucas is busy with another project today. Unfortunately this approach won't work since the SIDEBAR_MIN_WIDTH constant sets not only the sidebar's minimum width, but also its initial width when the user first runs Shotwell. With this change, the sidebar on a new Shotwell install is too narrow to see all dates when events are expanded. The right way to fix this is to modify the layout code so that the sidebar's default width is different from its minimum width. I don't think it's worth addressing this for 0.13 at this point. #### #5 Updated by Jens . 2 months ago Is this perhaps worth another look? It should be possible to set the sidebar to a smaller width (and all event or tag names should ideally overflow with "..." at the end or the font should be horizontally compressed, so that no horizontal scroll bar appears. #### #6 Updated by Jim Nelson 2 months ago * **Category** set to _ux_ One thing that should be noted (it wasn't true when the patch was reviewed) is that the sidebar can now be hidden with View -> Sidebar (F9). --- Bug imported by chaz@yorba.org 2013-11-25 21:57 UTC --- This bug was previously known as _bug_ 5214 at http://redmine.yorba.org/show_bug.cgi?id=5214 Imported an attachment (id=262428) Unknown version " in product shotwell. Setting version to "!unspecified". Unknown milestone "unknown in product shotwell. Setting to default milestone for this product, "---". Setting qa contact to the default for this product. This bug either had no qa contact or an invalid one. Resolution set on an open status. Dropping resolution
*** Bug 742812 has been marked as a duplicate of this bug. ***
With commit 25ed5e, sidebar items are ellipsized at the end. So even with long event names, it is now possible to have a thin sidebar. In addition, completely hiding the sidebar is also available as mentioned above. I think we could close this issue.
Good catch!
Created attachment 300381 [details] [review] patch With the new headers only sidebar, I think it really makes sense to lower the minimum sidebar width a bit. In addition I removed the SIDEBAR_MAX_WIDTH from the code, it was used nowhere. The initial width is not set by SIDEBAR_MIN_WIDTH but from Config.Facade.SIDEBAR_MIN_POSITION, so we don't have a problem there.
Review of attachment 300381 [details] [review]: Commit!
Pushed to master, commit a91a734
*** Bug 748851 has been marked as a duplicate of this bug. ***