-
Notifications
You must be signed in to change notification settings - Fork 16
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
opening external links does not open url in contained tab anymore #68
Comments
If it helps: not reproducible here with 0.0.65 on Firefox 56.0.1 on FreeBSD-CURRENT. (Do we have a shared understanding of 'external link'?) As a test:
Tentatively reading this alongside |
no, with external link I mean: having a contained tab open and then clicking a link in an external program (e.g. email client) -- this link used to be opened in the same container as the last active tab. |
Weird, I thought that I had posted more here. Probably drafted and (session storage) saved but lost through a crash that spoiled the store :-(
Whilst I can't recall exactly what was lost here, part of it involved post 10 (2017-05-21) under Move tab from container or to container - Test Pilot / Containers - Mozilla Discourse … |
Oh hurrah, I found the earlier draft, in a distant tab. Below, and I'm gonna click 'Comment' without checking for completeness … want to not lose track of this again …
OK, that's reproducible. I had no experience of prior behaviour. For what it's worth, the e-mail client use case is within https://discourse.mozilla.org/t/-/14134/10?u=grahamperrin with (I reckon) this keyword, quoted:
– then I rambled about potential for cumbersomeness. Is non-contained by default good, or bad? Right now, I can't say … Hey, today with this issue 68, by coincidence, clicks on a seemingly harmless – given the single click in Thunderbird that sort of led me to the clear drag-and-drop case, I'm toying with the possibility of edge case pollution of results, from other testers, where tabs in Firefox are added as a result of other external methods (e.g. not limited to drag-and-drop). |
17ecccb is trying to fix the bug ... I will run it for a few days and check how it feels. Will release 0.5.0 to github only as I have to play around with it for a few days -- feel free to install 0.5.0 from github and report back how it feels. |
should be connected to the changes yesterday :/
The text was updated successfully, but these errors were encountered: