Closed Bug 713008 Opened 10 years ago Closed 10 years ago

Tabs shouldn't flicker when hovered

Categories

(Thunderbird :: Toolbars and Tabs, defect)

x86
macOS
defect
Not set
normal

Tracking

(thunderbird11 fixed)

RESOLVED FIXED
Thunderbird 12.0
Tracking Status
thunderbird11 --- fixed

People

(Reporter: florian, Assigned: florian)

References

Details

Attachments

(1 file, 1 obsolete file)

Attached patch Hack (obsolete) — Splinter Review
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
Attachment #583837 - Flags: review?(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
Attachment #583837 - Flags: review?(mconley) → review+
(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.
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
Attached patch Patch v2Splinter Review
(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.
Assignee: nobody → florian
Attachment #583837 - Attachment is obsolete: true
Attachment #586938 - Flags: review?(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.
Attachment #586938 - Flags: review?(mconley) → review+
Comment on attachment 586938 [details] [diff] [review]
Patch v2

We'll want this for TB 11.
Attachment #586938 - Flags: approval-comm-aurora?
Committed to comm-central as http://hg.mozilla.org/comm-central/rev/439e89ac3f1d
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 12.0
Attachment #586938 - Flags: approval-comm-aurora? → approval-comm-aurora+
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.
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.
You need to log in before you can comment on or make changes to this bug.