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

Command requesting a new popup window with a container creates a regular window #4959

Open
asteroidcow opened this issue Mar 30, 2024 · 1 comment
Labels

Comments

@asteroidcow
Copy link

  • Brief description of the problem:

When using a command to run winopen() with -popup and -c Foo, the -popup flag is ignored.

  • Steps to reproduce:

Invoke the following command

command work jsb ['office.com','slack.com'].reduce((prev, url) => prev.then(() => tri.excmds.winopen('-popup', '-c', 'Work', 'https://' + url)), Promise.resolve());
  • Tridactyl version = 1.24.0
  • Firefox version = 124.0.1
@asteroidcow asteroidcow changed the title A new popup window with a container creates a regular window Command requesting a new popup window with a container creates a regular window Mar 30, 2024
@bovine3dom
Copy link
Member

Good spot, thanks.

Firefox doesn't support opening new windows with containers so our current workaround is to make a tab in a container and then detach it.

return tabopen(...args).then(() => tabdetach())

In theory, pop-up windows don't support tabs but we can hack it in by merging tabs into them. So

  1. make a popup window
  2. make a tab with the container in the current window
  3. merge that tab into the popup window
  4. close the original popup window tab

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

No branches or pull requests

2 participants