New tabs in wrong panel #2

mbnuqw / sidebery

Firefox extension for managing tabs and bookmarks in sidebar.

MIT License

3.26k stars 163 forks source link

New tabs in wrong panel #2 #126

Closed dzejba closed 4 years ago

dzejba commented 4 years ago

Steps to reproduce

Open Sidebery side strip in Firefox Select some Panel in Sidebery e.g. default one Open new tab by Firefox - either. Ctrl-T or click on "+" right to Firefox tabs

Expected behavior

Tab should be opened in current Sidebery panel

Actual behavior

Tab is opened in different Sidebery panel (the last created?) and Sidebery activates this Panel

Environment info

OS: Windows 10 Firefox version: 72.0.1 Addon version: 4.2.0

oliva commented 4 years ago

Same here, discovered through slightly different steps:

Open page on Default panel
Open page in second panel
Switch back to Default panel
Press Control-T to open a tab

New tab opens in second panel, Sidebery switches back to it

Versions: OS: linux Firefox: 74.0a1 Addon: 4.2.0

mbnuqw commented 4 years ago

Thanks for the report, soon will upload the new version with the fix.

mbnuqw commented 4 years ago

I've just uploaded v4.3.0, so you can try it. Also, check setting 'Position of new tab (for the other cases)'.

mpdeimos commented 4 years ago

Have sadly the same problem with 4.3.0. I'm wondering if there is any option I miss that controls where ctrl+T (new tab) opens. The only way I get this is moving the default panel as last position.

I think one possibility for an option would be making the default container explicit in the "Move tab of container to panel")

mbnuqw commented 4 years ago

@mpdeimos you're right, in c999b67 I've added default container to possible options, so I hope this issue and #125 will be resolved in the next release.

mpdeimos commented 4 years ago

Thanks. Will test as soon it's released!

ianelletson commented 4 years ago

Could there be multiple containers per panel? For example I have an "environments" panel which I'd like to add all my "local", "SIT", "QE", etc containers to automatically. I know I could do this with just 1:1 panel:container but I'd prefer 1:*

mbnuqw commented 4 years ago

@ianelletson, yes, this sounds reasonable, I think I can add possibility to select multiple containers for auto-move new tabs.

mbnuqw commented 4 years ago

@mpdeimos, v4.4.0 is released with the 'default panel' option, so you can try it! @ianelletson' I've created issue for your suggestion: #131

And I'm closing this issue