|
1.72 KB,
patch
|
neil@parkwaycc.co.uk
:
review+
|
Details | Diff | Splinter Review |
Created attachment 523098 [details] [diff] [review] patch Bug 533125 added support for browser.tabs.closeWindowWithLastTab and took care about the menuitem under File (cf. navigator.js). What's missing is middle clicking and the tab context menu, which still disables Close Tab if the pref is set to false.
|
||
Comment 1•6 years ago
|
||
(In reply to comment #0) > Bug 533125 added support for browser.tabs.closeWindowWithLastTab and took care > about the menuitem under File (cf. navigator.js). What's missing is middle > clicking and the tab context menu, which still disables Close Tab if the pref > is set to false. The tab close button always closes the tab, never the window, irrespective of the pref, and I think that middle-clicking should have the same behaviour. I agree that disabling close tab on the context menu is wrong, but as yet I'm not convinced that we actually need to disable it at all.
|
(Assignee) | |
Comment 2•6 years ago
|
||
(In reply to comment #1) > The tab close button always closes the tab, never the window, irrespective of > the pref, and I think that middle-clicking should have the same behaviour. OK, valid point. Didn't think of the tab bar close button; it's just too far away to be useful to me so I tend to forget about its existence. ;-) > I agree that disabling close tab on the context menu is wrong, but as yet I'm > not convinced that we actually need to disable it at all. I think we do because otherwise people may accidentally close the whole application (in case only a browser window is open) while expecting to close just the tab. Basically one of these cases that might become a trap for non-power users. [The alternative would be to rename the context menu item in that case but I don't like that idea.]
|
|
(Assignee) | |
Updated•6 years ago
|
||
|
|
||
Comment 3•6 years ago
|
||
(In reply to comment #2) > (In reply to comment #1) > > I agree that disabling close tab on the context menu is wrong, but as yet I'm > > not convinced that we actually need to disable it at all. > I think we do because otherwise people may accidentally close the whole > application (in case only a browser window is open) while expecting to close > just the tab. The close tab context menuitem would never do that; like the close button it only knows how to close tabs.
|
|
(Assignee) | |
Comment 4•6 years ago
|
||
Created attachment 523426 [details] [diff] [review] always enable Close Tab [Checkin: comment 6] Well, then: one-liner.
|
|
||
Comment 5•6 years ago
|
||
Comment on attachment 523426 [details] [diff] [review] always enable Close Tab [Checkin: comment 6] BTW Ctrl+F4 also always works if the tab bar is visible.
|
|
(Assignee) | |
Updated•6 years ago
|
||
|
|
(Assignee) | |
Comment 6•6 years ago
|
||
Comment on attachment 523426 [details] [diff] [review] always enable Close Tab [Checkin: comment 6] http://hg.mozilla.org/comm-central/rev/e1f6155dfa8d
|
|
(Assignee) | |
Updated•6 years ago
|
||
Description
•