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

Can't load a local HTML file with querystring via WebView.src (WebView.url works but is removed) #3872

Closed
EddyVerbruggen opened this issue Mar 24, 2017 · 3 comments · Fixed by #3980
Assignees
Labels
Milestone

Comments

@EddyVerbruggen
Copy link
Contributor

As noted here I'd like to request to not remove the WebView.url property as it's the only way to currently load a local HTML file with query parameters.

Passing ~/file.html to WebView.src works, but ~/file.html?foo=bar loads a blank page because it can't find the local file. WebView.url works perfectly for that case.

WebView.src can be altered to make it load the file successfully, but still I see no way in the current implementation to pass querystring params to the file. I really need a way to access those params in the loaded HTML file.

@NickIliev
Copy link
Contributor

Related to this fix.
Should we provide a way to pass query params via src or roll-back the url property?

Ping @vakrilov , @hshristov ^^^

@speigg
Copy link
Contributor

speigg commented Apr 5, 2017

IMO, I think we should keep both properties:

src should be the url that was explicitly set on the webview (and triggers a load)
url should be only a getter that always provides the current url (automatically updated on redirects, navigation, etc)

@vchimev vchimev added this to the 3.0 milestone May 1, 2017
@lock
Copy link

lock bot commented Aug 28, 2019

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked and limited conversation to collaborators Aug 28, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants