Thunderbird 60: always starts up showing Account Central pane
Categories
(Thunderbird :: Mail Window Front End, defect)
Tracking
(thunderbird_esr6869+ verified, thunderbird70 fixed, thunderbird71 fixed)
People
(Reporter: walts48, Assigned: aceman)
References
()
Details
(Keywords: regression)
User Story
Attachments
(1 file, 5 obsolete files)
1.25 KB,
patch
|
aceman
:
review+
jorgk-bmo
:
approval-comm-beta+
jorgk-bmo
:
approval-comm-esr68+
|
Details | Diff | Splinter Review |
Comment 1•7 years ago
|
||
Updated•7 years ago
|
Comment 2•7 years ago
|
||
Reporter | ||
Comment 3•7 years ago
|
||
Comment 4•7 years ago
|
||
Reporter | ||
Comment 5•7 years ago
|
||
Comment 9•6 years ago
|
||
Walt, can you test nightlies to find a one day regression range where this got worse?
(I realize the behavior isn't a regression, but the "severity" is)
Reporter | ||
Comment 10•6 years ago
|
||
55.0a1 Build ID: 20170501030205 is good.
Built from https://hg.mozilla.org/comm-central/rev/4d0138629df2a1b053836b59d50c33dc7d48fbbc
55.0a1 Build ID: 20170502030225 is the first nightly to display Account Central with the Inbox selected on startup..
Built from https://hg.mozilla.org/comm-central/rev/a7d9e926d66e7d79b2292945d7897cb5c9efe553
Not sure why I didn't see it in 56.0b4 Comment 5.
Comment 11•6 years ago
|
||
Excellent. That gives us https://hg.mozilla.org/comm-central/pushloghtml?fromchange=4d0138629df2a1b053836b59d50c33dc7d48fbbc&tochange=a7d9e926d66e7d79b2292945d7897cb5c9efe553
So bug 1358906 introduces a performance regression?
Comment 12•6 years ago
|
||
(In reply to Wayne Mery (:wsmwk) from comment #11)
Excellent. That gives us https://hg.mozilla.org/comm-central/pushloghtml?fromchange=4d0138629df2a1b053836b59d50c33dc7d48fbbc&tochange=a7d9e926d66e7d79b2292945d7897cb5c9efe553
So bug 1358906 introduces a performance regression?
No, in fact the opposite. Making it async means the window appears faster and paints the default view AC, while fetching the session file. Fixing the brief AC flash would mean waiting for the file read, but would be worth it as ui flash is bad. It could also possibly be fixed with css trickery.
Comment 13•6 years ago
|
||
So the effect is "psychological":
Before: Sync wait for the session file, then display the last session state.
Now: Display A/C (although unwanted), when async result comes back, display the last session state.
Comment 14•6 years ago
|
||
Yep. All recent animations and jank reduction are tricks on the user. In fact, merely having a throbber/progressbar will trick the user into not feeling UI is slow, even though something pops up with the same delay without them.
I don't see this on linux with any of the standard 3 layouts, but do on win7.
Comment 15•6 years ago
|
||
This affects my extension MoreLayouts with widethread view on linux, so.. This patch seems to work for that view, but should be built on win. If you can give it a try jorg.
Comment 16•6 years ago
|
||
Comment on attachment 9036208 [details] [diff] [review]
await.patch
Thanks for the patch, this looks good. Unfortunately my slow local debug build doesn't show the problem, when I start, I'm immediately in the last folder opened, no A/C in sight.
Richard, you do local opt builds, can you please check the patch, in case you usually see A/C when you start.
Comment 17•6 years ago
|
||
Comment on attachment 9036208 [details] [diff] [review]
await.patch
I don't see a difference. I've never seen the account central before the thread pane. Tested also on different VM's all I see is a white or grey background, depending on the OS, before the thread pane appears. And this hasn't changed with the patch.
Comment 18•6 years ago
|
||
OK, I'll try it myself then with a TB 60 ESR build:
https://treeherder.mozilla.org/#/jobs?repo=try-comm-central&revision=edab1ab67aed170d3a5cde92f51988b47db2f41a
Comment 19•6 years ago
|
||
Comment on attachment 9036208 [details] [diff] [review]
await.patch
I hate to hand out f-, but I tried it on the ESR and A/C still flashes up at startup. Just briefly, but just as much as "ordinary" ESR 60. I'll report again on a "cold start" after a reboot of the system in the morning.
Comment 20•6 years ago
|
||
if it doesn't work, it doesn't work ;)
can you try this one (added await in the main onload function).
Comment 21•6 years ago
|
||
move verifyAccounts() to ensure gSummaryFrameManager is created.
Comment 22•6 years ago
|
||
removing -purgecaches makes this more obvious on linux; the latest patch doesn't completely fix it either (starting up with vertical view).
Comment 23•6 years ago
|
||
You still want me to try? Linting changed all JS files on trunk in mail/, so I have to rebase the patch every time for 60 ESR before shipping it off to try :-(
Comment 24•6 years ago
|
||
If you can provide me Windows installer with a patch and i would be able to upgrade to an official version later (if not, i can maybe do a backup of my profile and revert after testing), i can test this on Windows 7 x64. I can reproduce the issue currently.
Comment 25•6 years ago
|
||
No, it isn't fixed for me, and doesn't seem like an easy fix.
Updated•6 years ago
|
Comment 28•6 years ago
|
||
Parts of this patch are in Bug 1554237 to fix a different thing, it's hard to tell if this problem is solved..
![]() |
Assignee | |
Comment 29•6 years ago
|
||
The problem still exists, it is even confirmed by bug 1568440, trying to solve the AC being redrawn twice (but that bug is probably for the case where the Account central is intended to stay displayed as an account is selected).
In my test in bug 1579575, the account central is still called twice on startup, once without any folder selected, then with the right selection.
We can catch the case where there is nothing selected, but that is a valid state when there are no accounts yet (new profile) and we need to distinguish that from the case where the folder tree just wasn't yet built (or session restored).
Comment 30•6 years ago
|
||
Yep, still present in 68.
![]() |
Assignee | |
Comment 31•6 years ago
|
||
What about a quick hack?
Comment 32•6 years ago
|
||
Comment 33•6 years ago
|
||
![]() |
Assignee | |
Comment 34•6 years ago
|
||
Yes I tested the final result of what stays displayed. But I didn't see the original bug in the first place, probably on a debug build it is so slow, that the session restore is done before the paint of the main window is shown. So I didn't see the flashing of AC then thread pane.
Walt, do you have a test profile to check a build of TB 71 ?
Reporter | ||
Comment 35•6 years ago
|
||
(In reply to :aceman from comment #34)
Yes I tested the final result of what stays displayed. But I didn't see the original bug in the first place, probably on a debug build it is so slow, that the session restore is done before the paint of the main window is shown. So I didn't see the flashing of AC then thread pane.
Walt, do you have a test profile to check a build of TB 71 ?
I still see a short flash of Account Central with an accounts Inbox, other folder, a feed or newsgroup selected.
I just updated my 71.0a1 to Build ID 20190911095508 Mozilla/5.0 (X11; Linux x86_64; rv:71.0) Gecko/20100101 Thunderbird/71.0a1 Ubuntu 18.04.3 LTS.
![]() |
Assignee | |
Comment 36•6 years ago
|
||
We haven't landed this in trunk, so try the build with the patch from here: https://queue.taskcluster.net/v1/task/KFmv1gngTzCNy-6hfBTUyA/runs/0/artifacts/public/build/target.tar.bz2 . Thanks.
Reporter | ||
Comment 37•6 years ago
|
||
(In reply to :aceman from comment #36)
We haven't landed this in trunk, so try the build with the patch from here: https://queue.taskcluster.net/v1/task/KFmv1gngTzCNy-6hfBTUyA/runs/0/artifacts/public/build/target.tar.bz2 . Thanks.
That one looks good with a new test profile, one email and newsgroup account created with several quits and restarts.
![]() |
Assignee | |
Comment 38•6 years ago
|
||
Comment 39•6 years ago
|
||
Comment 40•6 years ago
|
||
I'll land it later tonight and fix the comment.
![]() |
Assignee | |
Comment 41•6 years ago
|
||
Thanks, fixed the comment.
Comment 42•6 years ago
|
||
Pushed by mozilla@jorgk.com:
https://hg.mozilla.org/comm-central/rev/f8986e7f6f2e
Do not load Account Central until we have a folder selected on startup. r=mkmelin
Comment 43•6 years ago
|
||
Comment 44•6 years ago
|
||
TB 70 beta 2:
https://hg.mozilla.org/releases/comm-beta/rev/4d812d0a85aca34312b73ffec795818186b3d484
Updated•6 years ago
|
Comment 45•6 years ago
|
||
TB 68.1.1 ESR:
https://hg.mozilla.org/releases/comm-esr68/rev/73214b3f677012ca0e5a5f15f9ce614f13ae9905
Can someone please check that this works better now.
Comment 46•6 years ago
|
||
v.fixed 68.1.1 per walt
Comment 47•6 years ago
|
||
Confirm, that this is fixed in 68.1.1
Description
•