1353428 - Add pref to always open a new tab next to the current one to SeaMonkey

Status: RESOLVED FIXED

(SeaMonkey :: Tabbed Browser, enhancement)

Description

[[e-mail]]

+++ This bug was initially created as a clone of Bug #528005 +++

With many tabs, FF currently opens links or new tab somewhere at the far end of all tabs. Especially for links that are related to the current page, that's very inconvenient as user needs to drag newly opened tab all the way back to place it beside the current tab. There should be some way of opening links/new tabs right next to the current tab.

As implemented in FF, this was done as a modifier-click on the "+" button that adds a tab; I would greatly prefer that this be implemented as a preference, so that command-T in Mac OS and its Windows equivalent could be set to open the new tab to the right by default.

Comment 1

[[e-mail]]

(Looking at the part quoted in OP from  Bug #528005, I'm actually fine with the current behavior when one clicks on a link: in both FF and SM, one has the option of letting right-clicked links open immediately to the right. But I would definitely like to be able to set it so that wholly new tabs would also open to the right, whether opened with command-click or with the "New Tab" button).

Comment 2

[AndroidX]

I agree, opening a new tab next to the current one via CTRL+T keyboard shortcut would be very convenient. It doesn't need to be the default behavior, it could be controlled through a preference such as browser.tabs.insertNewAfterCurrent (as suggested in <https://bugzilla.mozilla.org/show_bug.cgi?id=528005#c18> 7 years ago).

Comment 3

[pisoir]

Same here. After > 7 years of waiting, please make this happen.

Comment 4

[bblack]

I have found a way how this can be solved easily.

There will be a new preference "browser.tabs.insertAllTabsAfterCurrent": default = false.

If "browser.tabs.insertAllTabsAfterCurrent" = true, then all new tabs are inserted in the same way.
If "browser.tabs.insertRelatedAfterCurrent" = false, then all new tabs are insert at the end, just like before.
If "browser.tabs.insertRelatedAfterCurrent" = true, then all new tabs are insert right next to the current tab.

All means: 
open a link in new tab, 
open a new tab with the "New Tab" button, 
open a new tab via CTRL+T keyboard shortcut,
tab is opened from an external application or bookmark or ....

Comment 5

[bblack]

Hi Frank, can you please take over these two lines in SM2.53

browser-prefs.js:

+ pref("browser.tabs.insertAllTabsAfterCurrent", false);

tabbrowser.xml Line 1685:

- if ((aRelatedToCurrent || aReferrerURI) && this.mPrefs.getBoolPref("browser.tabs.insertRelatedAfterCurrent")) {

+ if ((aRelatedToCurrent || aReferrerURI || this.mPrefs.getBoolPref("browser.tabs.insertAllTabsAfterCurrent")) &&   this.mPrefs.getBoolPref("browser.tabs.insertRelatedAfterCurrent")) {

These two lines make it easier for many people who use many tabs.

If you do not change the prefs, the behavior of SM does not change.
if you change browser.tabs.insertAllTabsAfterCurrent to true under about:config, every new tab will be added right next to the current tab.

Comment 6

[Frank-Rainer Grahl (:frg)]

This should work but why don't you just add a patch and asking for review?

I reset the flags. target is only set  to the current version (2.60 now) after review and checkin. This is an old enhancement bug. If the patch gets r+ we should decide later in which to push it. Might well go into a future 2.49 too. No l10n changes here.

Comment 7

[bblack]

Do I have to install Mercurial to upload the patch?
Is there an easy way to add a small patch?

Comment 8

[bblack]

Attachment: patch-1353428.diff

I hope the format of the patch is correct.

Comment 9

[Frank-Rainer Grahl (:frg)]

Comment on attachment 8999474 [details] [diff] [review]
patch-1353428.diff

Sorry I am a bit busy this week. Might manage Thursday.

The patch is missing a correctly formatted header. If you install at least mozilla-build 3.2 and Tortoise Hg you could use Mercurial queues which makes life much easier if you plan to dive into the code.

Comment 10

[Frank-Rainer Grahl (:frg)]

Attachment: 1353428-newtab.patch

bblack, I reformatted the patch and tested it in both 2.53 and 2.57. Wasn't aware that you just diffed the unpacked source until I tried to appy it. I am asking IanN for a final review because it introduces some new functionality via a pref.

f+ from me.

If you do other patches please install mercurial.

Comment 11

[Ian Neal]

Comment on attachment 9002254 [details] [diff] [review]
1353428-newtab.patch

r/a=me

Comment 12

[Ian Neal]

As the patch doesn't quite add the functionality in the subject, can the subject be amended and a follow-up bug spun off?

Comment 13

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/7fc7bd15790d
Add option to open new tabs always next to the current one. r=IanN

Comment 14

[Frank-Rainer Grahl (:frg)]

Comment on attachment 9002254 [details] [diff] [review]
1353428-newtab.patch

Works fine in 2.53 and just needs a mini rebase for the prefs for 2.49 so I think lets take it to 2.49.5

Comment 15

[Frank-Rainer Grahl (:frg)]

comm-central is still 2.60

Comment 16

[Ian Neal]

Comment on attachment 9002254 [details] [diff] [review]
1353428-newtab.patch

a=me

Comment 17

[Frank-Rainer Grahl (:frg)]

That was fast :) 

Target 2.49.5
https://hg.mozilla.org/releases/comm-esr52/rev/7bf4aec57fa4d7e466c5f547ad9e4bdfa37fafdd

Comment 18

[Frank-Rainer Grahl (:frg)]

Forgot to post the esr60 rev:

https://hg.mozilla.org/releases/comm-esr60/rev/22886572a3c2