Created attachment 8374695 [details] Tab Overflow I have several windows with tabs filling up and overflowing the tab bar. On most windows the far right tab is selected. After restoring a session, *all* of my windows whose tab rows are full and that have the rightmost tab selected are scrolled ever so slightly off the tab bar - the bottom curve it seems (see attached screenshot). I tried reproducing this with a different profile (filling several windows with tabs) and ironically could only get the *leftmost* selected tab to trail off the end, like the screenshot, on session restore. It's obviously not a showstopper, but it's poor polish and for someone like me with that tinge of OCD I have to go through and hit the scroll arrow on all the cases. It just looks bad.
Hm, I could reproduce this last week but can't anymore. Caspy7, can you still reproduce this on a new nightly build?
(In reply to Jared Wein [:jaws] (please needinfo? me) from comment #2) > Hm, I could reproduce this last week but can't anymore. Caspy7, can you > still reproduce this on a new nightly build? Still seeing this on the latest nightly.
Ok, I can reproduce with the left-most selected tab.
I'm pretty sure I've gotten it figured out now. Patch coming soon.
Created attachment 8402890 [details] [diff] [review] Patch Kinda stinks to reach in to private methods and properties of mTabstrip, but it's not the first place within tabbrowser.xml.
Created attachment 8402892 [details] [diff] [review] Patch v1.01 (forgot to qref)
Comment on attachment 8402892 [details] [diff] [review] Patch v1.01 (forgot to qref) I agree on both the gross-ness of sucking out private members of mTabstrip, and also at the sentiment that "we were doing it already, so what the hell." We really should consider trying to expose those as public read-only members instead. I wouldn't block on it, but can you please file a bug to do that? Also, have you done a try push to see how this could affect TART? If not, maybe do that. Or not, and take your chances on fx-team / m-c. :)
Compare-talos shows that there is no TART regression from this patch. Landed on fx-team: https://hg.mozilla.org/integration/fx-team/rev/6aef8ef7d775
Comment on attachment 8402892 [details] [diff] [review] Patch v1.01 (forgot to qref) [Approval Request Comment] Bug caused by (feature/regressing bug #): introduction of tab curves missed this case that has to be manually adjusted for the negative margins on tab curves User impact if declined: session restore can have partial tabs showing (curve is clipped) Testing completed (on m-c, etc.): on m-c for a few days Risk to taking this patch (and alternatives if risky): none expected String or IDL/UUID changes made by this patch: none
Aurora: https://hg.mozilla.org/releases/mozilla-aurora/rev/2200d43e5485 Beta: https://hg.mozilla.org/releases/mozilla-beta/rev/9798420b26fc
Caspy7, can you try and see if this works now for you with the builds below? http://ftp.mozilla.org/pub/mozilla.org/firefox/nightly/29.0b8-candidates/build1/ http://ftp.mozilla.org/pub/mozilla.org/firefox/nightly/latest-mozilla-aurora/ http://ftp.mozilla.org/pub/mozilla.org/firefox/nightly/latest-mozilla-central/
I tested with all builds and the problem appears resolved in all of them.