Closed Bug 255932 Opened 20 years ago Closed 20 years ago

Add "Warn when closing multiple tabs" pref to Advanced under Browsing

Categories

(Firefox :: Tabbed Browser, enhancement)

enhancement
Not set
normal

Tracking

()

RESOLVED FIXED
Firefox1.0beta

People

(Reporter: bugzilla, Assigned: steffen.wilberg)

References

Details

(Keywords: fixed-aviary1.0)

Attachments

(1 file)

Steffen, do you think this is something you'd have time to do? This pref already exists, it's browser.tabs.warnOnClose. It's just a matter of adding a checkbox under Browsing in Advanced. We'd have to do this by PR to get it in.
Attached patch patchSplinter Review
Comment on attachment 156384 [details] [diff] [review] patch Here you are :)
Attachment #156384 - Flags: review?(firefox)
Severity: normal → enhancement
Status: NEW → ASSIGNED
Flags: blocking-aviary1.0PR?
OS: Windows XP → All
Hardware: PC → All
Target Milestone: --- → Firefox1.0beta
Attachment #156384 - Flags: review?(firefox)
Attachment #156384 - Flags: review+
Attachment #156384 - Flags: approval-aviary+
Flags: blocking-aviary1.0PR? → blocking-aviary1.0PR+
Checked in br & trunk.
Keywords: fixed-aviary1.0
*** Bug 255260 has been marked as a duplicate of this bug. ***
Status: ASSIGNED → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
R.J., Jeff, we need to update the Help docs on this.
I added this to TBP a while back; when 1.0 is released I'll probably remove the redundant UI. Although the message "Warn when closing multiple tabs" seems a little vague. I chose the wording "Warn on closing a window with multiple tabs open".
*** Bug 255967 has been marked as a duplicate of this bug. ***
Bradley, this warning applies to closing a window with multiple tabs and to the "Close other tabs" command, so the pref has to be worded to include both.
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: