1389883 - Thunderbird 56.0.beta2 Titlebar corrupted on MAC

Status: RESOLVED FIXED

(Thunderbird :: Theme, defect)

Description

[Ian Hamilton]

Attachment: Screen Shot 2017-08-13 at 15.24.18.png

User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.12; rv:54.0) Gecko/20100101 Firefox/54.0
Build ID: 20170628075643

Steps to reproduce:

Default install


Actual results:

Inbox tab sits behind Maximize, Minimize, Full screen buttons


Expected results:

Inbox tab sits beside Maximize, Minimize, Full screen buttons

Comment 1

[Richard Marti (:Paenglab)]

Hmm, same as bug 1389110.

Ian, you upgraded from 55 to 56? I don't know of any change in this area. Could you try to go into the Preferences/Advanced/General and click on "Config Editor". Then in the search field enter 'drawintitlebar', now double click it. There should be now the titlebar with the controls above the tabs. Now double click the line again. Are the controls correctly separated again?

If not, could you try bug 1389110 comment 5?

Comment 2

[Ian Hamilton]

I seem to recall that the problem occurred with version 55 also
Changing the value of "drawintitlebar" to false dropped the tabs below the window buttons
but changing it back to false caused the problem to re-occur.
Re: Bug 1389110 Comment 5
There was no localstore.json
Deleting xulstore.json made no difference when xulstore.json was recreated

Comment 3

[Richard Marti (:Paenglab)]

I don't see this problem, so it's a digging in the dark for me. Can you try to restart with Add-ons disabled (see help menu)? If this also doesn't help, can you start TB and then type CMD + SHIFT + J? Are there errors which could cause this issue? You can select and copy them.

Comment 4

[Ian Hamilton]

Attachment: Screen Shot 2017-08-13 at 21.31.03.png

Screenshot of error log

Comment 5

[Ian Hamilton]

Starting safe mode makes no difference

Comment 6

[Richard Marti (:Paenglab)]

Attachment: Bug1389883.patch

I was wrong with no change in this area. :(

Thanks to the console log I found a typo. Strange I don't see this on my Mac.

Ian, I started a try build. When it's finished, I'll give you the link to test it, okay? It will be a 57 Daily.

Comment 7

[Jorg K (CEST = GMT+2)]

Comment on attachment 8896732 [details] [diff] [review]
Bug1389883.patch

I'll fix the commit message to:
Bug 1389883 - Bug 1385852 follow-up: Fix typo in Mac code part. r=jorgk

https://hg.mozilla.org/comm-central/rev/045fa1d535b5#l1.292

Comment 8

[Richard Marti (:Paenglab)]

Ian, please could you test https://archive.mozilla.org/pub/thunderbird/try-builds/[e-mail]-36b5d673ff09aa01c4f1de36e74e864e4fc7ae06/try-comm-central-macosx64/ ?

Comment 10

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/17dbc1c8a88b
Bug 1385852 follow-up: Fix typo in Mac code part. r=jorgk

Comment 11

[Richard Marti (:Paenglab)]

Comment on attachment 8896732 [details] [diff] [review]
Bug1389883.patch

Bug was opened from beta. If there is a plan for a beta, this should land also in beta.

Comment 12

[Ian Hamilton]

I downloaded the file "thunderbird-57.0a1.en-US.mac.dmg" but could not open it.
It contained "daily.app" but macos (10.12.6) said the file was damaged and could not be opened

Comment 13

[Richard Marti (:Paenglab)]

Ian, thanks for trying it. The error message is misleading. The app isn't signed and macOS blocks it because of this. And I can't say how I resolved this to execute it anyway (Mac is not my main platform). It has something to do with the security settings. But with the error in the console it was clear what the problem was. I hope there will be another 56 beta that you can test.

Comment 14

[Ian Hamilton]

at command prompt type:

sudo spctl --master-disable

This allows app from anywhere.

I ran your version and it does indeed work. I notice that the tab shape is now square.
Is that the default going forward or just a characteristic of this beta?

Comment 15

[Jorg K (CEST = GMT+2)]

Thanks for testing. The square tabs will go live in TB 57 beta, you've been testing an "alpha" version, TB 57 Daily. That follows the UI restructuring in Firefox, please read here:
https://dolske.wordpress.com/2017/08/04/photon-engineering-newsletter-11/

Comment 16

[Jorg K (CEST = GMT+2)]

Beta (TB 56):
https://hg.mozilla.org/releases/comm-beta/rev/93cec452e327bb21236f5c8bb8df109b9bd3b623