344958 - Add hidden preference for turning off tab jumpback

Status: VERIFIED FIXED

(Camino Graveyard :: Tabbed Browsing, enhancement)

Description

[Chris Lawson (gone)]

Set your prefs to:

Cmd-click opens a new tab
New windows/tabs open in background

Cmd-click a few of the links on the above URL (although it shouldn't be in any way specific to that site). Then Cmd-shift-click a link -- it opens in a new foreground tab.

OK, that's fine. Now close that tab.

In trunk builds, you jump back to the tab in which you clicked. In 1.0.x builds, the *previous* tab (immediately to the left) is activated.

Unless I missed something, this was not a desired behaviour change and is definitely a regression from 1.0.x. I'm pretty sure this happened sometime in the last month, but right now the smallest window I can give is "since 1.0 branched off".

Comment 1

[Samuel Sidler (old account; do not CC)]

Isn't this just tab jumpback? You're jumping back to the tab you came from?

Comment 2

[froodian (Ian Leue)]

This is expected behavior with tab jumpback.  However, I believe that pink verbally endorsed a hidden pref for turning off jumpback, so I'm morphing this bug to that.

Comment 3

[froodian (Ian Leue)]

Attachment: Patch

Creates camino.enable_tabjumpback and sets it to true by default.

Comment 4

[Håkan Waara]

The question is if we want prefs (hidden or not) for every behavior change we make...

Comment 5

[Smokey Ardisson (offline for a while; not following bugs - do not email)]

Have people complained about this?  (After the bugs I filed on it were fixed, that is?)  I really think that the behavior we have really is done just right (since those bugs were fixed).

Comment 6

[froodian (Ian Leue)]

Well, cl's use case seems like a pretty reasonable one: Open a list of links in the background, open the last of the list in the foreground, then work your way back.

We haven't received many complaints, but the majority of our user-base hasn't ever used a build with jumpback yet.  If people really think it's an unreasonable feature, we can wait for alpha/beta releases and see if we get complaints, but I think wanting to turn off jumpback is a reasonable wish.

Comment 7

[Mike Pinkerton (not reading bugmail)]

i don't think we want a ui pref for this. we need to get the behavior correct and stick with it. a hidden pref......well ok, but it should just work. My daily-use build doesn't have jumpback and god it's annoying now that i know the difference.

Comment 8

[Mike Pinkerton (not reading bugmail)]

Comment on attachment 229541 [details] [diff] [review]
Patch

sr=pink

Comment 9

[Nick Kreeger]

Fixed trunk and branch.