Closed
Bug 1379268
Opened 7 years ago
Closed 7 years ago
Switching between Firefox windows caused tab background color to lag behind titlebar
Categories
(Firefox :: Theme, defect, P1)
Firefox
Theme
Tracking
()
Tracking | Status | |
---|---|---|
firefox-esr52 | --- | unaffected |
firefox54 | --- | unaffected |
firefox55 | --- | unaffected |
firefox56 | --- | fixed |
firefox57 | --- | verified |
People
(Reporter: benjamin, Assigned: dao)
References
(Depends on 1 open bug, Blocks 1 open bug)
Details
(Keywords: regression, Whiteboard: [photon-visual][p1])
Attachments
(2 files)
After bug 1196266, switching between Firefox windows with the mouse causes a pretty serious mismatch between the titlebar and the tab background for a period of time.
Screencast attached.
This is more serious when the titlebar is showing because you can see the delay visually. When the titlebar is off (the default), the delay is still there but Firefox just feels slow rather than showing a visual mismatch.
Comment 1•7 years ago
|
||
I can't reproduce on my Windows VM, but that doesn't mean other people aren't seeing this. This is a pretty serious problem.
Whiteboard: [photon-visual] → [photon-visual][triage]
Updated•7 years ago
|
Assignee | ||
Comment 2•7 years ago
|
||
(In reply to Benjamin Smedberg [:bsmedberg] from comment #0)
> This is more serious when the titlebar is showing because you can see the
> delay visually.
The solution here will be to not apply the accent color beyond the title bar when the native title bar is enabled.
> When the titlebar is off (the default), the delay is still
> there but Firefox just feels slow rather than showing a visual mismatch.
There's probably not much we can do about that.
Assignee: nobody → dao+bmo
Flags: qe-verify+
Priority: -- → P1
Whiteboard: [photon-visual][triage] → [photon-visual][p1]
Reporter | ||
Comment 3•7 years ago
|
||
> There's probably not much we can do about that.
Why not? Can't we at least profile and figure out why focus events aren't coming through very quickly?
Assignee | ||
Comment 4•7 years ago
|
||
(In reply to Benjamin Smedberg [:bsmedberg] from comment #3)
> > There's probably not much we can do about that.
>
> Why not? Can't we at least profile and figure out why focus events aren't
> coming through very quickly?
That's of course always a good idea, and people have in fact profiled window activation. Bug 1334642 was filed and fixed as a result. I just suspect there aren't more low-hanging fruits left.
Updated•7 years ago
|
Status: NEW → ASSIGNED
Iteration: --- → 56.2 - Jul 10
QA Contact: brindusa.tot
Comment 5•7 years ago
|
||
The other option would be to draw the titlebar as part of the Firefox content instead of using the Windows native titlebar. Then the whole window would switch delayed, but at least it would switch at the same time.
Updated•7 years ago
|
status-firefox54:
--- → unaffected
status-firefox55:
--- → unaffected
status-firefox56:
--- → affected
Updated•7 years ago
|
Iteration: 56.2 - Jul 10 → 56.3 - Jul 24
Comment hidden (mozreview-request) |
Comment 7•7 years ago
|
||
I don't understand why we're not using the dark blue background for Photon here. Can you clarify that?
Flags: needinfo?(dao+bmo)
Assignee | ||
Comment 8•7 years ago
|
||
(In reply to Johann Hofmann [:johannh] from comment #7)
> I don't understand why we're not using the dark blue background for Photon
> here. Can you clarify that?
Having two different and dominant colors up there doesn't sound like a good idea to me. It's too colorful and jarring.
Flags: needinfo?(dao+bmo)
Comment 9•7 years ago
|
||
mozreview-review |
Comment on attachment 8885263 [details]
Bug 1379268 - Don't apply the accent color beyond the title bar when the native title bar is enabled.
https://reviewboard.mozilla.org/r/156132/#review161546
Ok, to me it sounds like the whole UI concept is getting a bit too colorful/inconsistent if the tab bar is grey, but I don't want to stall on a subjective matter. We can always change the color later.
Attachment #8885263 -
Flags: review?(jhofmann) → review+
Comment 10•7 years ago
|
||
Pushed by dgottwald@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/a426a96fbedb
Don't apply the accent color beyond the title bar when the native title bar is enabled. r=johannh
Comment 12•7 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 56
Updated•7 years ago
|
status-firefox-esr52:
--- → unaffected
Updated•7 years ago
|
QA Contact: brindusa.tot → ovidiu.boca
Comment 13•7 years ago
|
||
Verified on Windows 10 x 64 on the latest Nightly 57.0a1 ID 20170816100153. The issue is not reproducible. I will mark this bug as verified fixed. Thank you
Updated•7 years ago
|
Assignee | ||
Updated•7 years ago
|
You need to log in
before you can comment on or make changes to this bug.
Description
•