-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Feature: set browser path #1448
Conversation
Thanks for raising this PR and updating the documentation. @stffabi any thoughts? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you such much @NanoNik for putting your time into this PR. I only have some minor comments which I think should be addressed before it's getting merged. Overall it looks good to me.
v2/internal/frontend/desktop/windows/go-webview2/pkg/edge/chromium.go
Outdated
Show resolved
Hide resolved
v2/internal/frontend/desktop/windows/go-webview2/pkg/edge/chromium.go
Outdated
Show resolved
Hide resolved
…for installed webview in manual strategy
Head branch was pushed to by a user without write access
Would it be possible to use |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Thanks for all your effort you have put into this 🚀
Please fix merge conflicts and we should be good 👍 |
Resolves #1446
This PR adds new webview2 strategy: manual
It won't download or notify user about webview2 installation status.
You must manually specify a path to a webview2 runtime in
windows.Options
structure.If the specified directory does not exist, it will try to use webview2 installed in the system.