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

Implement firefox container tab support #263

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

whi-tw
Copy link

@whi-tw whi-tw commented Apr 13, 2022

Firefox can have container tabs enabled, and an extension exists that will allow routing external URLs with a specific format into these containers Open External links in a container.

This change implements logic so if configured in a handler, the outgoing URL is converted into this format when the browser is called.

This has an improvement over using a rewrite rule, in that the config can be kept DRY, and container selection logic is left to the handler. An example of such a rewrite rule implementation is here: #211 (comment).

Example configuration:

handlers: [
  {
    // Some work site
    match: "https://some-site.com/*",
    browser: {
      name: "Firefox",
      container: "Work",
    },
  },
],

Firefox can have container tabs enabled, and an extension exists
that will allow routing urls with a specific format into these
containers [Open External links in a container](https://addons.mozilla.org/en-GB/firefox/addon/open-url-in-container/).

If configured in a `handler`, rewrite the outgoing URL into this format
and open that.

This has an improvement over using a rewrite rule, in that the config
can be kept DRY, and container selection logic is left to the handler.

Example configuration:

```js
handlers: [
  {
    // Some work site
    match: "https://some-site.com/*",
    browser: {
      name: "Firefox",
      container: "Work",
    },
  },
],
```
@whi-tw whi-tw marked this pull request as ready for review April 13, 2022 12:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant