Skip to content

Feature: set browser path#1448

Merged
leaanthony merged 12 commits intowailsapp:masterfrom
NanoNik:feature/setBrowserPath
Jun 24, 2022
Merged

Feature: set browser path#1448
leaanthony merged 12 commits intowailsapp:masterfrom
NanoNik:feature/setBrowserPath

Conversation

@NanoNik
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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.

Comment thread v2/internal/frontend/desktop/windows/go-webview2/pkg/edge/chromium.go Outdated
Comment thread website/docs/reference/options.mdx
Comment thread v2/internal/frontend/desktop/windows/go-webview2/pkg/edge/chromium.go Outdated
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
Copy Markdown
Member

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

Comment thread v2/internal/wv2installer/manual.go Outdated
@NanoNik NanoNik requested review from leaanthony and stffabi June 22, 2022 21:20
Copy link
Copy Markdown
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
Copy Markdown
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