713008 - Tabs shouldn't flicker when hovered

Status: RESOLVED FIXED

(Thunderbird :: Toolbars and Tabs, defect)

Description

[Florian Quèze [:florian]]

Attachment: Hack

The first time a non-selected tab is hovered, it flickers because the tab-hover-active.png image isn't loaded yet.

The attached patch reproduces the hack Firefox uses to work-around this: http://mxr.mozilla.org/mozilla-central/source/browser/themes/pinstripe/browser.css#1885

Comment 1

[Mike Conley (:mconley) (:⚙️)]

Comment on attachment 583837 [details] [diff] [review]
Hack

Review of attachment 583837 [details] [diff] [review]:
-----------------------------------------------------------------

Florian:

This looks fine to me - and thanks for filing / tackling these issues!

Does this apply to gnomestripe / qute as well?  If so, it's an r+, but you should update the patch to stretch across the other two platforms as well.

Thanks,

-Mike

Comment 2

[Florian Quèze [:florian]]

(In reply to Mike Conley (:mconley) from comment #1)

> Does this apply to gnomestripe / qute as well?  If so, it's an r+, but you
> should update the patch to stretch across the other two platforms as well.

I hadn't checked at the time I attached the patch (I was mostly interested in getting rid of the flickering that kept attracting my eyes today while frequently restarting Thunderbird to debug something else), but this is the only place where your Tabs-on-top patch adds an image that's visible only on hover.

Comment 3

[Mike Conley (:mconley) (:⚙️)]

Florian:

Is this patch supposed to fix the black-box flash thing in the tab selector on start up?  Because I'm still seeing it with this patch...

Also, the patch has bitrotted slightly.

-Mike

Comment 4

[Florian Quèze [:florian]]

Attachment: Patch v2

(In reply to Mike Conley (:mconley) from comment #3)
> Florian:
> 
> Is this patch supposed to fix the black-box flash thing in the tab selector
> on start up?  Because I'm still seeing it with this patch...
> 
> Also, the patch has bitrotted slightly.

I don't see the the black-box on startup, but only on unselected tabs. My previous patch fixed only the flashing when hovering an unselected tab for the first time, but I've noticed it also happens when opening a tab in the background (during my initial testing, I already had 2 tabs at startup), this new patch fixes this case too, and also unbittrots.

Comment 5

[Mike Conley (:mconley) (:⚙️)]

Comment on attachment 586938 [details] [diff] [review]
Patch v2

Review of attachment 586938 [details] [diff] [review]:
-----------------------------------------------------------------

Looks good, Florian!

Just two nits.  Modulo those, r=me.

Thanks,

-Mike

::: mail/themes/pinstripe/mail/tabmail.css
@@ -107,4 +106,5 @@
> >  .tabmail-tab:not([selected="true"]):hover {
> >    -moz-border-image: url(tabs/tab-hover-active.png) 4 5 3 6 fill repeat stretch;
> >  }
> >  
> > +/* preloading hack */

Could you quickly document what the preloading hack does, and why we need it?

@@ +108,5 @@
>  }
>  
> +/* preloading hack */
> +#tabs-toolbar::after {
> +   content: '';

The contents of this block should be 2 spaces in, not 3.

Comment 6

[Mike Conley (:mconley) (:⚙️)]

Comment on attachment 586938 [details] [diff] [review]
Patch v2

We'll want this for TB 11.

Comment 7

[Mike Conley (:mconley) (:⚙️)]

Committed to comm-central as http://hg.mozilla.org/comm-central/rev/439e89ac3f1d

Comment 8

[Mike Conley (:mconley) (:⚙️)]

So, I was a bit silly, and I committed this to trunk without letting Florian fix the nits I found.

I've made the corrections I pointed out, and committed to comm-aurora here:

http://hg.mozilla.org/releases/comm-aurora/rev/f03fea92e7fc

and we'll need a follow-up bug to fix the nits in comm-central.  Forthcoming.

Comment 9

[Mike Conley (:mconley) (:⚙️)]

So it turns out that Florian *did* fix the nits before they were committed as http://hg.mozilla.org/comm-central/rev/439e89ac3f1d.  I think we're good then.