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

Add support for brave-browser as chrome-bin #68

Open
bsless opened this issue Oct 12, 2021 · 8 comments
Open

Add support for brave-browser as chrome-bin #68

bsless opened this issue Oct 12, 2021 · 8 comments
Labels
enhancement New feature or request

Comments

@bsless
Copy link
Contributor

bsless commented Oct 12, 2021

Hello,
I'm starting to use Portal and encountered a slight inconvenience, it does not recognize Brave browser
Brave is another chromium based browser which can probably be used just like the other executables found by get-chrome-bin, and supports launching in app mode.
The linux executable is called brave-browser, not sure regarding other operating systems.

Thanks

@bsless
Copy link
Contributor Author

bsless commented Oct 12, 2021

As a temporary workaround I symlinked chromium to brave

@djblue
Copy link
Owner

djblue commented Oct 12, 2021

@bsless This sounds like a good inclusion. Would you mind submitting a PR with the proposed change?

@djblue djblue added the enhancement New feature or request label Oct 12, 2021
@bsless
Copy link
Contributor Author

bsless commented Oct 12, 2021

@djblue Gladly, though I don't have OSX or Windows so I won't be able to test it there, but I'll give it a best effort.
Also, would you like to add any action or setting for default browser? A user might have one installed but prefer portal opens with the other.

@djblue
Copy link
Owner

djblue commented Oct 12, 2021

No worries, I can test out those platforms. That's a good point about different browser choices. What do you think the option should look like?

@bsless
Copy link
Contributor Author

bsless commented Oct 13, 2021

open can take an option for preferred browser which would be looked for first, :portal.launcher/browser?
In the same vain, an option to save these configuration and have portal automatically search for them would also be nice. in *nix systems you can place a file in XDG_CONFIG_PATH, with windows it's always a riddle. Then users won't need to specify browser, color, or other preferred options.

@bsless
Copy link
Contributor Author

bsless commented Oct 13, 2021

Thinking some more about browsers, especially if you want to make it "future proof", it might be a good idea to have some data spec for browsers and how portal consumes them. Things like the executable name on each operating system, extra paths to search in, supported operating systems, web engine, etc.
That might make life easier when the next user wants to add specific support for Edge, Opera, or a new browser (optimistic, I know)

@yannvanhalewyn
Copy link

I'm really enjoying Portal :). I would also like something like this: I would like to be able for Portal to open in Arc browser on Mac, which is another Chromium based browser. Now I let it open in Chrome, copy the URL and close Chrome.

@djblue
Copy link
Owner

djblue commented Dec 23, 2024

@yannvanhalewyn The chrome stuff is mostly for app mode. If you just want to have Portal open in your default browser, you could do (portal.api/open {:app false}) which will use clojure.java.browse/browse-url.

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

No branches or pull requests

3 participants