PikachuEXE / tabkit

Automatically exported from code.google.com/p/tabkit
0 stars 0 forks source link

Context-menu search opens in new tab outside of group to the left. #149

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
== What exact steps will reproduce the problem? ==

1. Highlight any text on a web page in a tab group.
2. Left-click for context menu.
3. Right-click "Search".

== What is the expected output? What do you see instead? ==

When launching a search from the context menu from within a group, a new search 
results tab should open to the right of the current tab and it should not get 
switched to automatically.

Instead, the new search results tab opens _initially_ to the right of the 
current tab then immediately gets bounced out of the group all the way to the 
far left.  The new tab gets switched to immediately.  If the left edge of the 
group is off the screen, the tab bar only _sometimes_ scrolls to this new tab 
with the search results.

== What versions of Tab Kit and Firefox are you using? On what operating
system? ==

TabKit 2nd Edition 0.9.9.8
GNU/Linux 2.6.32-5-686

== Please provide any additional information below. ==

Annoyed.

Original issue reported on code.google.com by jshe...@gmail.com on 5 Jan 2013 at 9:03

GoogleCodeExporter commented 9 years ago
Oops, using Firefox 17.0.1

Original comment by jshe...@gmail.com on 5 Jan 2013 at 9:05

GoogleCodeExporter commented 9 years ago
A workaround is to select "Treat blank new tabs as related to the current tab"  
under Advanced options.

However, _all_ new tabs will be related to the current group, and not just the 
search results tab... and it still gets switched to immediately... which 
becomes problematic when you wish to do multiple internet searches with various 
terms on any given page.

Perhaps a new Advanced option under "When grouping tabs by opener:" like "Treat 
new context-menu search tabs as related to the current tab" and "Don't 
immediately switch to it"

Original comment by jshe...@gmail.com on 5 Jan 2013 at 9:39