search

bwinton / TabCenter

Firefox add-on for arranging tabs vertically
https://testpilot.firefox.com/experiments/tab-center
209 stars 55 forks source link

Pinning Breaks layout in 1.9 #53

Closed johngruen closed 8 years ago

johngruen commented 8 years ago

In 1.9 (54a16d217d6) Pinning overlaps the browser chrome and creates a big blank space on the right hand side of the browser: screen shot 2016-04-18 at 11 13 15 am

jaredhirsch commented 8 years ago

Looks like an easy fix to me: when the tabcenter is pinned, #appcontent (the thing that holds the visible web page) needs to have its left value overridden, from the default 45px to 260px. Easy to verify this, just give it a try in the Browser Toolbox.

bwinton commented 8 years ago

Yeah, but why didn't it do that before? (Also, I suspect we want to change #panel-3-1, so that the DevTools and Findbar are also scrolled over.)

Dammit. #rbwoser[tabspinned="true"] #verticaltabs-box + #appcontent { :P