Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

"Keep old tabs" should not keep useless tabs (new tab page, temp tabs) #93

Closed
kristofferR opened this issue Sep 13, 2019 · 8 comments · Fixed by #144
Closed

"Keep old tabs" should not keep useless tabs (new tab page, temp tabs) #93

kristofferR opened this issue Sep 13, 2019 · 8 comments · Fixed by #144

Comments

@kristofferR
Copy link

kristofferR commented Sep 13, 2019

The "Keep old tabs" functionality is super confusing. It works as it says, but I'd like it to work more intelligently. Zombie temp tabs are left when links are ctrl-clicked. If I open a new empty tab and enter a containerized site it opens a new tab with the page, but still keeps the old useless new tab page.

If I enter a new URL in the address bar, it just opens a new tab with that page instead of replacing the current one.

I'd much rather have it work like this:

  • Empty New tab pages should not be kept
  • Temp tabs redirected to a new container should not be kept
  • Manually input URLs should overwrite the current site

Here's a video of the first two issues:
https://streamable.com/0etkx

@kristofferR kristofferR mentioned this issue Sep 13, 2019
16 tasks
@ghost
Copy link

ghost commented Sep 13, 2019

It came from #14 . I honestly didn't think it was a good idea, hence the preference and not default behavior. Personally, I won't be working on this directly. The next big feature (after default containers) will be exit rules for containers which might include this.

PRs are welcome though.

@go2null
Copy link

go2null commented Oct 23, 2020

this is what prevents me from using the Keep old tabs option - as it lead to too much manual cleaning up.

@switz
Copy link
Contributor

switz commented Nov 12, 2020

Yeah, this is relatively easily fixable UX. I'm happy to implement this if we get confirmation it'll be merged.

switz added a commit to switz/containerise that referenced this issue Nov 24, 2020
switz added a commit to switz/containerise that referenced this issue Nov 24, 2020
switz added a commit to switz/containerise that referenced this issue Nov 24, 2020
@muzzah
Copy link

muzzah commented Dec 26, 2020

@switz I also experience this but Im not sure your change will fix it. I have the default container option enabled and also regex to open google domain in a separate google container. If I am on a google container and then click on a link (which does not match any regex i have setup thus should open a default container linked tab) that opens a new tab, that new tab is still linked to the google container and then when it goes to load the actual (non google) link, it spawns another tab which loads the page but that initial tab it it opened is still there with a google link.

So lets say I visit google shopping (in google container), find an item and click visit site, I end up with 3 tabs, 1 google shopping (google container), 1 google redirect link (Google container with google redirect url but is a blank page), 1 actual website (Default container).

@switz
Copy link
Contributor

switz commented Dec 26, 2020

I don't use a default container so that may be an edge case that this doesn't solve.

What is the google redirect page URL? After that intermediate tab is created, open your developer console and type window.location.href. What url is it actually?

@muzzah
Copy link

muzzah commented Dec 28, 2020

When you click the visit site link, it opens a URL in the form of "https://www.google.com/aclk?sa...." in a new tab (So google shopping uses _blank type attribute) as to keep the shopping website open. This then redirects to a new URL which opens in a new default container tab.

You can go to the shopping.google.com website, search for a then click on the visit site link for that product to replicate.

@muzzah
Copy link

muzzah commented Dec 28, 2020

ok interesting to note, when I do use the console to check the value you mention, it shows about:blank

@switz
Copy link
Contributor

switz commented Dec 29, 2020

Great! It should work flawlessly for you then; that's the same as me.

kintesh added a commit that referenced this issue Feb 5, 2021
Remove temporary tabs if keepOldTabs is true. Fixes #93
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants