[e10s] Tab separator flashes when opening a new tab

RESOLVED INACTIVE

Status

()

Firefox
Theme
P5
normal
RESOLVED INACTIVE
3 years ago
2 days ago

People

(Reporter: phlsa, Unassigned)

Tracking

(Blocks: 1 bug)

Trunk
Unspecified
All
Points:
---

Firefox Tracking Flags

(e10s+, firefox44 affected)

Details

(Whiteboard: [qx:spec])

Attachments

(1 attachment)

Created attachment 8680569 [details]
Screen_Shot_2015-10-29_at_11_40_52.png

When opening a new tab, the tab separator briefly flashes and overlays the new tab button. It shouldn't do that.

Tested on OS X and Windows 10.
Can you still reproduce this? I just tried, and I don't see it, on either beta or current nightly (tested on OS X and Windows 8.1)
Flags: needinfo?(philipp)
I was still able to reproduce it in todays Nightly on Windows 10 but it seems to be really dependent on how busy the machine is. Looks like there are a few frames when the overlapping happens, and when the machine is busy, those are the frames that freeze quite often.
Flags: needinfo?(philipp)
Priority: -- → P5
Rakhi, can you reproduce this on your windows 10 machine if you try a few times?
Flags: needinfo?(Rakhish1994)

Comment 4

2 years ago
Yes! I am able to reproduce it.
Flags: needinfo?(Rakhish1994)

Comment 5

2 years ago
So, I am able to reproduce the bug, but still not able to find out how to fix this tab separator overlap. I noticed few things while trying:

1. I was not able to inspect it because overlap appears for a second only. I try to increase transition time so that I can inspect but no! overlap still appears for a second.
2.The selector that depends on [visuallyselected] not being set here, which is causing the separator to get shown here. This is different in e10s vs non-e10s because that attribute doesn't get set immediately. By disabling e10s, tab separator overlap doesn't happened(probably because in e10s we switch tabs asynchronously).

Any idea about how fix this?
Flags: needinfo?(dao+bmo)
Sounds like more fallout from bug 1009628.
Flags: needinfo?(dao+bmo) → needinfo?(mconley)
Summary: Tab separator flashes when opening a new tab → [e10s] Tab separator flashes when opening a new tab
I wonder if it makes sense to hide the tab separators before and after a tab that is animating? That'd maybe sidestep this a little.

Can you think of any problems with that solution, shorlander?
Flags: needinfo?(mconley) → needinfo?(shorlander)
(In reply to Mike Conley (:mconley) - (Needinfo me!) from comment #7)
> I wonder if it makes sense to hide the tab separators before and after a tab
> that is animating? That'd maybe sidestep this a little.
> 
> Can you think of any problems with that solution, shorlander?

Then you can't see the tab at all, right? You'd only see the [+] button moving with no separator before it. It'd also be weird for e.g. closing tabs in the middle of other tabs - you'd see a continuous blurb of the text of 3 tabs with no separators between them for a bit. Am I misunderstanding what you're proposing, Mike?

Orthogonally, it doesn't really make sense to me that the tab separator ever displays *on top of* the new tab button. That shouldn't even be 'physically' possible in terms of the margins/paddings involved. Feels like we're missing something.
Flags: needinfo?(mconley)
(In reply to :Gijs Kruitbosch from comment #8)
> Then you can't see the tab at all, right? You'd only see the [+] button
> moving with no separator before it. It'd also be weird for e.g. closing tabs
> in the middle of other tabs - you'd see a continuous blurb of the text of 3
> tabs with no separators between them for a bit. Am I misunderstanding what
> you're proposing, Mike?

Yeah, that's accurate - I think my idea was a little half-baked.

> 
> Orthogonally, it doesn't really make sense to me that the tab separator ever
> displays *on top of* the new tab button. That shouldn't even be 'physically'
> possible in terms of the margins/paddings involved. Feels like we're missing
> something.

Do we know that the separator is actually appearing on top of the newtab button? The screenshot doesn't make it super clear - since (from what I remember) the new tab button has a transparent background (and just shows an outline when hovered), perhaps we're seeing a tab separator leaking through that transparency?
Flags: needinfo?(mconley)
(In reply to Mike Conley (:mconley) - (Needinfo me!) from comment #9)

> Do we know that the separator is actually appearing on top of the newtab
> button? The screenshot doesn't make it super clear - since (from what I
> remember) the new tab button has a transparent background (and just shows an
> outline when hovered), perhaps we're seeing a tab separator leaking through
> that transparency?

I made the new tab button background opaque and I still the separator appearing. So it appears to be on top of it.
Flags: needinfo?(shorlander)

Updated

2 years ago
tracking-e10s: --- → -

Updated

2 years ago
tracking-e10s: - → +

Comment 11

2 days ago
Per policy at https://wiki.mozilla.org/Bug_Triage/Projects/Bug_Handling/Bug_Husbandry#Inactive_Bugs. If this bug is not an enhancement request or a bug not present in a supported release of Firefox, then it may be reopened.
Status: NEW → RESOLVED
Last Resolved: 2 days ago
Resolution: --- → INACTIVE
You need to log in before you can comment on or make changes to this bug.