Graphic anomalies on "Account Central"
Categories
(Thunderbird :: General, defect, P2)
Tracking
(thunderbird_esr78 fixed, thunderbird78 wontfix, thunderbird79 wontfix, thunderbird80 fixed)
People
(Reporter: jorgk-bmo, Assigned: aleca)
References
Details
(Keywords: regression)
Attachments
(2 files, 1 obsolete file)
16.00 KB,
image/png
|
Details | |
5.15 KB,
patch
|
Paenglab
:
review+
wsmwk
:
approval-comm-beta+
wsmwk
:
approval-comm-esr78+
|
Details | Diff | Splinter Review |
See attached image.
I also noticed that windows for Email, Address Book, etc. open centre screen, but Calendar doesn't.
Alex, did you implement this?
Assignee | ||
Comment 1•3 years ago
|
||
Mhhh, it seems that the email tooltip remained visible, probably a focus issue, I'll take a look.
I'll see why some dialogs open at the center of the screen and others at the center of the application.
Assignee | ||
Updated•3 years ago
|
Assignee | ||
Comment 2•3 years ago
|
||
This removes the showing of the button description on focus, since we can't control the focus ring if a user clicks on a button and then hovers over another button, the 2 descriptions will show at the same time.
Let's remove it for safety.
I set centerscreen
for the Calendar dialog, but I don't think it'll fix it since some dialogs are modals, and some aren't, so putting centerscreen will sometimes open at the center of TB, or at the center of your screen, and I don't think I can't control that.
I'm launching a 78 try-run for you to test, Jorg.
Assignee | ||
Comment 3•3 years ago
|
||
Here's a 78 Windows try-run (still building at the time of this message): https://treeherder.mozilla.org/#/jobs?repo=try-comm-central&revision=82c7557ee6b07ae24e883e5f28be3e2cb0cb3724
Updated•3 years ago
|
Comment 4•3 years ago
|
||
Comment on attachment 9166427 [details] [diff] [review] 1655289-account-central.diff The tooltips don't overwrite theme any more, good. But the Calendar dialog opens still on top left. What happens here: - Email centre screen - Calendar top left. But it saves the position. When moved to an other position it opens the there. - Address Book centre TB - Chat centre TB - Feeds centre TB -Newsgroups centre TB We should look that they all open at the same position, preferably centre TB. On a big screen, I have a 3440 * 1440 screen, the centre screen and the centre TB can be on totally other positions.
Assignee | ||
Comment 5•3 years ago
|
||
Now all the dialogs should open as modals at the center of TB.
A little heads up about this issue reported 5 years ago, which I can't reproduce, so I think we can assume we can drop the workaround: https://searchfox.org/comm-central/rev/306ca78c3d2d1b7a247d670c26d2dd719b8df4da/calendar/base/modules/utils/calWindowUtils.jsm#29
Assignee | ||
Comment 6•3 years ago
|
||
Just to leave a note somewhere, this will need a separated 78 variation as it doesn't merge normally.
I'll take care of it after it gets an r+
Comment 7•3 years ago
|
||
Comment on attachment 9166649 [details] [diff] [review] 1655289-account-central.diff Thanks, looks good now. I don't see Bug 1151440 on my Ubuntu Linux, so removing this workaround should be okay.
Assignee | ||
Updated•3 years ago
|
Assignee | ||
Comment 8•3 years ago
|
||
Comment on attachment 9166649 [details] [diff] [review] 1655289-account-central.diff [Approval Request Comment] Regression caused by (bug #): - User impact if declined: Conflict between hover and focus effect on Account Hub button description, and wizard dialogs not always opening at the center of the application. Testing completed (on c-c, etc.): soon on c-c Risk to taking this patch (and alternatives if risky): moderate as we removed some previous workaround for old (5+ years) Linux issues. I recommend keeping this on trunk and beta to be sure nothing was affected.
Comment 10•3 years ago
|
||
Pushed by alessandro@thunderbird.net:
https://hg.mozilla.org/comm-central/rev/355771da5346
Fix UI issues with tooltips and opening dialogs in new Account Central. r=Paenglab
Comment 11•3 years ago
|
||
Comment on attachment 9166649 [details] [diff] [review] 1655289-account-central.diff With the TM change, this needs uplift to beta 80 ... [Triage Comment] Approved for beta
Assignee | ||
Comment 12•3 years ago
|
||
I just flagged bug 1656360 for beta and esr uplift.
That needs to land together with this patch as it fixes a pretty annoying regression.
Comment 13•3 years ago
|
||
bugherder uplift |
Thunderbird 80.0b2:
https://hg.mozilla.org/releases/comm-beta/rev/e8d971daba35
Updated•3 years ago
|
Updated•3 years ago
|
Comment 14•3 years ago
|
||
Forgot the comment.
Tested in 80.0b2 on Ubuntu 18.04.4.
Comment 15•3 years ago
|
||
Comment on attachment 9166649 [details] [diff] [review]
1655289-account-central.diff
[Triage Comment]
Approved for esr78
Comment 16•3 years ago
|
||
bugherder uplift |
Thunderbird 78.1.2:
https://hg.mozilla.org/releases/comm-esr78/rev/91803303b24f
Description
•