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

Reordering containers in Manage Containers doesn't apply to other Firefox menus #1958

Closed
emyarod opened this issue Jan 31, 2021 · 8 comments
Closed
Assignees
Labels
bug Something is broken! Needs: Mozilla Central Needs changes in Mozilla-Central
Milestone

Comments

@emyarod
Copy link

emyarod commented Jan 31, 2021

  • Multi-Account Containers Version: 7.3.0
  • Operating System + Version: Windows 10, macOS, Linux
  • Firefox Version: 87 Nightly

Actual behavior

as reported by @hub2git in #330

After updating to Firefox 85.0 on Linux, I now can sort the containers -- as they appear in the toolbar -- by drag and drop.

But the changes are not reflected when right-clicking a link on a webpage and hovering over "Open Link in New Container Tab".

Expected behavior

order of containers in right click context menu should match order in the toolbar

Steps to reproduce

  1. reorder containers
  2. observe that the order of containers differs between the context menu and toolbar dropdown
@hub2git
Copy link

hub2git commented Jan 31, 2021

@emyarod thanks for creating this new issue and for mentioning me.

@jonathanKingston
Copy link
Contributor

Duplicate of: https://bugzilla.mozilla.org/show_bug.cgi?id=1333395

@hub2git

This comment was marked as off-topic.

@Twisted-Code

This comment was marked as off-topic.

@achernyakevich-sc
Copy link

I have decided to provide screenshot that visually explain the problem:

image

@Karl-MCS
Copy link

If you click the Multi-Account Containers toolbar button and click Manage Containers, you can re-order the containers manually by drag-and-drop. However, in all other menus (long-click/right-click + tab button, right-click tab or link and hover Open in New Container Tab) the containers retain their original unsorted order in which they were created.

@jlaundry
Copy link

So I had a quick look at this, link to WIP: main...jlaundry:multi-account-containers:main#files

Always open this in a Container (URL bar icon) is relatively easy - we can just create a getIdentities() function in Utils.js, replicate the behaviour from Logic.refreshIdentities(), and use this instead of a direct const identities = browser.contextualIdentities.query({})

Open in a New Container Tab (tab right-click menu) and Container Tabs (Settings page) seem to be tricker... I've tried brute-forcing it by find/replacing with Utils.getIdentities() everywhere, but that doesn't seem to have any effect. I suspect, given the l10n strings are in mozilla-central, that this may be outside this extension? But not sure (I'm just a Firefox fan, not usually a dev!)

@groovecoder
Copy link
Member

The fix for this has been released. #2672

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something is broken! Needs: Mozilla Central Needs changes in Mozilla-Central
Projects
None yet
Development

No branches or pull requests

10 participants