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

Added WebView::load_html #1058

Open
wants to merge 9 commits into
base: dev
Choose a base branch
from

Conversation

VictorKoenders
Copy link

@VictorKoenders VictorKoenders commented Nov 6, 2023

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Docs
  • Code style update
  • Refactor
  • Build-related changes
  • Other, please describe:

Does this PR introduce a breaking change?

  • Yes
  • No

Checklist

  • This PR will resolve #___
  • A change file is added if any packages will require a version bump due to this PR per the instructions in the readme.
  • I have added a convincing reason for adding this feature, if necessary
  • It can be built on all targets and pass CI/CD.

Other information

There is currently WebViewBuilder::with_html but there is no way to load in HTML after the WebView has been constructed. This is an attempt at adding that.

I have only tested this on Windows right now. The other platforms still need testing / fixing. Please chip in if you can test or have feedback.

@VictorKoenders VictorKoenders marked this pull request as ready for review November 6, 2023 14:32
@VictorKoenders VictorKoenders requested a review from a team as a code owner November 6, 2023 14:32
@dklassic
Copy link
Contributor

dklassic commented Nov 7, 2023

Tested on Android using cargo-mobile2, seems to be working properly.

@VictorKoenders
Copy link
Author

Rebased onto dev

@pewsheen
Copy link
Contributor

pewsheen commented Nov 7, 2023

Also tested in Windows/macOS/ubuntu

@amrbashir
Copy link
Member

@VictorKoenders looks like you didn't sign your commits and I can't merge the PR unless they are signed. You need to setup commit signing, then you can sign past commit like this for example.

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.

None yet

4 participants