1655289 - Graphic anomalies on "Account Central"

Status: VERIFIED FIXED

(Thunderbird :: General, defect, P2)

Description

[Jorg K (CEST = GMT+2)]

Attachment: 2020-07-26 00_19_03-BMO - Mozilla Thunderbird.png

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?

Comment 1

[Alessandro Castellani [:aleca]]

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.

Comment 2

[Alessandro Castellani [:aleca]]

Attachment: 1655289-account-central.diff

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.

Comment 3

[Alessandro Castellani [:aleca]]

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

Comment 4

[Richard Marti (:Paenglab)]

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.

Comment 5

[Alessandro Castellani [:aleca]]

Attachment: 1655289-account-central.diff

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

Comment 6

[Alessandro Castellani [:aleca]]

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

[Richard Marti (:Paenglab)]

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.

Comment 8

[Alessandro Castellani [:aleca]]

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 9

[Jorg K (CEST = GMT+2)]

81

Comment 10

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/355771da5346
Fix UI issues with tooltips and opening dialogs in new Account Central. r=Paenglab

Comment 11

[Wayne Mery (:wsmwk)]

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

Comment 12

[Alessandro Castellani [:aleca]]

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

[Rob Lemley [:rjl]]

Thunderbird 80.0b2:
https://hg.mozilla.org/releases/comm-beta/rev/e8d971daba35

Comment 14

[WaltS48 [:walts48]]

Forgot the comment.

Tested in 80.0b2 on Ubuntu 18.04.4.

Comment 15

[Wayne Mery (:wsmwk)]

Comment on attachment 9166649 [details] [diff] [review]
1655289-account-central.diff

[Triage Comment]
Approved for esr78

Comment 16

[Geoff Lankow (:darktrojan)]

Thunderbird 78.1.2:
https://hg.mozilla.org/releases/comm-esr78/rev/91803303b24f