Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

clicking offsite links in pinned tab should always open a new tab #569

Closed
asadotzler opened this issue Feb 2, 2016 · 9 comments
Closed

Comments

@asadotzler
Copy link

When I pin a tab, I want links to other sites to open in a new tab and only on-site links to open in the pinned tab. For example, if I pin gmail, I want to be able to navigate around gmail in that pinned tab but I want a link in an email to some other site to open in a new tab.

@bbondy
Copy link
Member

bbondy commented Feb 2, 2016

That's already how it's implemented, it will do same origin navigation within the tab but out of origin into a new tab.

I bet there's a use case we're missing though. Let me know the STR.

What I do is:

  1. Go to http://slashdot.org/
  2. Click on some of the links that lead to internal stories, I can navigate within the pin.
  3. Click on a link that goes to an external site (like in the news titles to the right it has a domain to another page link sometimes).

In step 2 I stay within the pinned tab.
In step 3 it opens a new tab.

@asadotzler
Copy link
Author

  1. Visit https://news.ycombinator.com/ and pin the tab.
  2. Click a link to a story
  3. See the off-site story load in the pinned tab.

Also the slashdot case fails for me too. Broken on Mac and Windows 0.7.12 build.

@luixxiul
Copy link
Contributor

luixxiul commented May 15, 2016

Also the slashdot case fails for me too.

Me too, on 0.9.6 (Windows 10).

@luixxiul
Copy link
Contributor

I cannot reproduce this on Ubuntu 14.04, so this might be specific to Windows.

@luixxiul
Copy link
Contributor

I think this has been already fixed. @bsclifton would you please confirm?

@bsclifton
Copy link
Member

bsclifton commented May 28, 2016

Confirmed, this is still an issue. Here's a bare-bones way to repro:

  1. Visit https://media.clifton.io/issue-569/2016-05-27-brave-pin.html
  2. Pin the site
  3. Click the "same domain" link, notice it works as expected (opens in same tab)
  4. Click back
  5. Click the "external link"
  6. Notice it didn't work; it opens in the same tab

slashdot was a little tricky since they have _blank on almost all of their external elements

@bsclifton
Copy link
Member

cc: @bradleyrichter - maybe you can point me to some docs on how pinned tabs should work.

I have a basic idea just from using them (and they seem VERY cool- almost a better version of a bookmarks toolbar). Any clicks which go to another domain should open in a new tab, right? (regardless of target being set or not- ex: _blank)

@bbondy
Copy link
Member

bbondy commented May 29, 2016

@bsclifton it's a bug, same origin should stay within the same pinned tab. Out of origin should move to a new tab.

@luixxiul luixxiul added bug and removed suggestion labels Nov 2, 2017
@bsclifton bsclifton added this to the Triage Backlog milestone Nov 27, 2017
@bsclifton bsclifton added wontfix fixed-with-brave-core This issue will automatically resolved with the replacement of Muon with Brave Core. labels Aug 30, 2018
@bsclifton bsclifton removed this from the Triage Backlog milestone Aug 30, 2018
@bsclifton bsclifton added open-in-brave-core and removed fixed-with-brave-core This issue will automatically resolved with the replacement of Muon with Brave Core. labels Aug 30, 2018
@bsclifton
Copy link
Member

Closing in favor of brave/brave-browser#882

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants