Open All In Tabs for bookmarks needing containers causes duplicate tabs
Categories
(Firefox :: Bookmarks & History, defect)
Tracking
()
People
(Reporter: jon+bugzilla.mozilla.org, Unassigned)
Details
Attachments
(1 file)
|
50.59 KB,
text/plain
|
Details |
User Agent: Mozilla/5.0 (X11; Linux x86_64; rv:95.0) Gecko/20100101 Firefox/95.0
Steps to reproduce:
- I have a bookmark folder containing a number of bookmarks (8), all of which are for hosts which are associated with a defined container
- I select "Open All In Tabs" from that folder in the bookmarks menu
Actual results:
Firefox opens 16 tabs, 2 per bookmark in the folder.
(My theory, and this is just a theory, is that it opens each bookmark in a new tab, which is the default container/not contained, then firefox determines it should be contained and so spawns a new tab accordingly, yet also replaces the current tab with a container tab as well.)
Expected results:
Firefox should open 8 tabs, one per bookmark.
|
||
Comment 1•3 years ago
|
||
The Bugbug bot thinks this bug should belong to the 'Firefox::Bookmarks & History' component, and is moving the bug to that component. Please revert this change in case you think the bot is wrong.
|
||
Comment 2•3 years ago
|
||
How do you associate sites to containers, do you use an add-on, or a different workflow?
I'm asking because we have various opens bugs about making bookmarks that open in a specific container, like bug 1545950 and bug 1502167.
I suspect this bug is mostly due to that, since we don't have internal support for opening bookmarks in containers, add-ons have to do hack-arounds to enforce that.
So, I first of all need to know which add-on you are using for containers, and then it may be useful if you could Attach a txt with contents from the about:support page that contains useful debug information.
|
Reporter | |
Comment 3•3 years ago
|
||
How do you associate sites to containers, do you use an add-on, or a different workflow?
A mixture of "Firefox Multi-Account Containers"-provided UI "Edit Site List" per container, and also patterns via "Containerise" (https://github.com/kintesh/containerise).
I'll try to create a minimal reproducer to narrow things down.
|
|
Reporter | |
Comment 4•3 years ago
|
||
|
|
||
Comment 5•3 years ago
|
||
Yes, so I think the problem here is our partial support for containers, we can't do much if we don't address that underlying issue, add-ons will have to workaround all the misses. Thus, I'm duping to the existing bugs because I don't see us introducing hacks on top of partial support.
Description
•