GNOME Bugzilla – Bug 731443
GtkPlacesSidebar - refine spacing
Last modified: 2014-08-13 18:36:05 UTC
Created attachment 278186 [details] modified screenshot During the 3.13.x cycle, the sub-headings were removed from GtkPlacesSidebar. This is good, and makes it much cleaner. However, the layout could still be improved, and there are a few small visual issues left by the change: * No padding between the top row and the top of the sidebar, which doesn't look great when the top item is selected. * With the headings gone, the list feels more visually constrained. * The lines which separate the groups add visual noise, and leave the groups seeming rather ambiguous - sections are delineated, but the user has to figure out what the lines mean. I've attached a modified screenshot, which shows how these issues could be resolved. The original screenshot is on the left, and the modified version is on the right. Changes: * Add 4px padding above the top list item, to give it clearance from the top of the sidebar. * Increase the row height by 4px. * Remove the separators.
Created attachment 278718 [details] [review] places-sidebar: Remove the separators and add more margin
Looks good to me!
<mclasen> I don't like the outright removal of separators left and right
(In reply to comment #3) > <mclasen> I don't like the outright removal of separators left and right Let's leave the separators in for now then.
Created attachment 283281 [details] [review] places-sidebar: Remove the separators and add more margin