1390108 - Port bug 1375335 to TB [Menu bar is too high]

Status: RESOLVED FIXED

(Thunderbird :: Theme, enhancement)

Description

[Richard Marti (:Paenglab)]

Bug 1375335 makes some changes in the item calculations for the titlebar/tabbar elements. We should change our code to be in sync with them.

Comment 1

[Richard Marti (:Paenglab)]

Attachment: controlCalculation.patch

This is a port of https://hg.mozilla.org/integration/autoland/rev/37ba4f932f57 which lands soon.

This is to test only with drawintitlebar=true

Without patch you see, when you have maximised the window and change to the normal window size, for a very short time a white line above the tabs toolbar and then all is jumping up 1px. With patch it doesn't happen.

We are differing in the .titlebar-button padding. We use 9px top/bottom-padding instead of 8px to let the window controls not jump in the height when showing/hiding the menu. This is also caused because of our different implementation with two toolboxes for the menubar/tabbar and the main toolbar. FX uses a margin-bottom: -1px on the tabbar we needed to solve differently because of our separated toolboxes. That's also the cause why I needed to make the tabbbar height even (30px).

Also for Win7 Aero we are different with the position of the menubar with below the tabs. Because of this, we can now remove a lot of code which added/removed margins.

Comment 2

[Jorg K (CEST = GMT+2)]

That's the one that landed: https://hg.mozilla.org/mozilla-central/rev/fdc6247f8fca

Comment 3

[Richard Marti (:Paenglab)]

Only the test changed to the one I noted.

Comment 4

[Jorg K (CEST = GMT+2)]

Comment on attachment 8896949 [details] [diff] [review]
controlCalculation.patch

I applied the patch and don't see any difference in looks on Windows 10.

(In reply to Richard Marti (:Paenglab) from comment #1)
> Without patch you see, when you have maximised the window and change to the
> normal window size, for a very short time a white line above the tabs
> toolbar and then all is jumping up 1px. With patch it doesn't happen.
It still happens. While before the "very short time" was quite noticeable, I measured 100 ms, it's now down to 40 ms. Maybe you don't see it any more on an opt build. On the Daily opt build without this patch, I don't see it.

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/89fef90ea52f
Port bug 1375335 to TB [Fix window control height calculation on Windows 10]. r=jorgk