Closed Bug 1133265 Opened 7 years ago Closed 7 years ago
.tabs .draw In Titlebar by default on OSX
Thunderbird looks so much nicer on mac with this enabled. Any reason we shouldn't do this for tb38?
OS: Mac OS X → All
Hardware: x86 → All
Version: 30 → Trunk
This only applies to Mac. Windows and Linux have the pref enabled already.
OS: All → Mac OS X
This was done in bug 941740. It actually looks good now, so I think we can just do it.
Assignee: nobody → philipp
Status: NEW → ASSIGNED
Attachment #8564680 - Flags: review?(josiah)
Comment on attachment 8564680 [details] [diff] [review] drawInTitlebar.diff As Linux doesn't support yet draw into titlebar it would be better to do this like Firefox: https://dxr.mozilla.org/comm-central/source/mozilla/browser/app/profile/firefox.js#451 to not enable it without needed code when it's possible.
Comment on attachment 8564680 [details] [diff] [review] drawInTitlebar.diff Review of attachment 8564680 [details] [diff] [review]: ----------------------------------------------------------------- Looks good if you apply Paenglab's feedback. There was a reason we were waiting to enable this, but I don't think we need to wait any longer. Thanks for writing the patch!
Attachment #8564680 - Flags: review?(josiah) → review+
Summary: Enable mail.tabs.drawInTitlebar by default → Enable mail.tabs.drawInTitlebar by default on OSX
Hmm ok, I've done as you asked. But looking at the diff, we had it enabled by default on Linux before, right? We only had it disabled for mac? This patch would be disabling it for Linux and enabling it for mac.
I think it's a no-op on Linux, so it doesn't really matter.
Yes, Linux doesn't support drawInTitlebar yet and it doesn't matter. But when it's support it it's better to disable it until everything is fixed.
Ok, as you wish. Setting checkin-needed for the updated patch that disables it on linux and enables it on mac.
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 38.0
You need to log in before you can comment on or make changes to this bug.