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

Feature: set browser path #1448

Merged
merged 12 commits into from
Jun 24, 2022
Merged

Conversation

NanoNik
Copy link
Contributor

@NanoNik NanoNik commented Jun 12, 2022

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.

@leaanthony
Copy link
Member

Thanks for raising this PR and updating the documentation. @stffabi any thoughts?

@leaanthony leaanthony enabled auto-merge (squash) June 12, 2022 22:50
Copy link
Collaborator

@stffabi stffabi left a 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.

auto-merge was automatically disabled June 15, 2022 18:28

Head branch was pushed to by a user without write access

@NanoNik NanoNik requested a review from stffabi June 15, 2022 20:58
@leaanthony
Copy link
Member

Would it be possible to use UTF16FromString rather than StringToUTF16Ptr as that call has been deprecated. Thanks 👍

Copy link
Collaborator

@stffabi stffabi left a 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 🚀

@leaanthony
Copy link
Member

Please fix merge conflicts and we should be good 👍

@leaanthony leaanthony enabled auto-merge (squash) June 24, 2022 19:14
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.

Ability to specify path to WebView2
3 participants