Closed Bug 1724515 Opened 3 years ago Closed 3 years ago

Allow mail tab bar to be controlled separately to browser tab bar

Categories

(SeaMonkey :: MailNews: Message Display, enhancement)

enhancement

Tracking

(seamonkey2.53+ fixed)

RESOLVED FIXED
seamonkey 2.90
Tracking Status
seamonkey2.53 + fixed

People

(Reporter: iannbugzilla, Assigned: iannbugzilla)

References

Details

(Whiteboard: SM2.53.10)

Attachments

(1 file)

At the moment browser.tabs.autoHide controls the tab bar in both browser and mailnews windows, we should have a separate pref to control the mail tab bar.

[Approval Request Comment]
Regression caused by (bug #): n/a
User impact if declined: Only combined control of tab bar
Testing completed (on m-c, etc.): 2.53.10
Risk to taking this patch (and alternatives if risky): low
String changes made by this patch: 2 new strings and changes to help

Attachment #9235206 - Flags: review?(frgrahl)
Attachment #9235206 - Flags: approval-comm-release?
Attachment #9235206 - Flags: approval-comm-esr60?
Summary: All mail tab bar to be controlled separately to browser tab bar → Allow mail tab bar to be controlled separately to browser tab bar

Comment on attachment 9235206 [details] [diff] [review]
1724515-mailAutoHide-25310.patch

LGTM

NIT Initial mail.tabs.autoHide should be false to match browser. Will change for checkin.

Do we need a migration routine to populate it intially with a user set browser.tabs.autoHide value?

Attachment #9235206 - Flags: review?(frgrahl)
Attachment #9235206 - Flags: review+
Attachment #9235206 - Flags: approval-comm-release?
Attachment #9235206 - Flags: approval-comm-release+
Attachment #9235206 - Flags: approval-comm-esr60?
Attachment #9235206 - Flags: approval-comm-esr60+
Blocks: 1729165

Pushed by frgrahl@gmx.net:
https://hg.mozilla.org/comm-central/rev/0908c6f5d500
All mail tab bar to be controlled separately to browser tab bar. r=frg

Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Whiteboard: SM2.53.10
Target Milestone: --- → seamonkey 2.90
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: