The toolbar needs to be taller in order to match the mockup . Currently the buttons are almost touching the borders. 1. http://25.media.tumblr.com/tumblr_m0iwtyy6nh1qkoea4o1_1280.png
Created attachment 624494 [details] [diff] [review] patch This patch give a min-height of 41px inclusive bottom border. The top border is from tabs-toolbar. This gives the 42px like on mockup. I centered the toolbar in the toolbox. when two toolbars are used this grows the toolbox only slightly instead of a 84px tall toolbox. I hope this is how it's planned to work. This patch works also for toolboxes in other tabs like Chat and Lightning. Because the same should be for OSX I expanded the patch also for this. See http://24.media.tumblr.com/tumblr_m0it9bMzM51qkoea4o1_1280.png Additionally I centered the OSX searchbox through giving the same top and bottom margin.
Comment on attachment 624494 [details] [diff] [review] patch Looks good ui and code-wise.
Fixed by Bug 763308
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
Whiteboard: Fixed by Bug 763308
You need to log in before you can comment on or make changes to this bug.