Skip to content
This repository has been archived by the owner on Feb 22, 2023. It is now read-only.

Add an initialUrl and a javaScriptMode parameters to WebView #753

Merged
merged 1 commit into from
Sep 6, 2018

Conversation

amirh
Copy link
Contributor

@amirh amirh commented Aug 31, 2018

Adding just these 2 parameters to set up the skeleton for webview
settings and initial parameters.

Settings (like javaScriptMode) are set and updated by rebuilding the
widget.

initialUrl is a covinience parameter for simple use cases, as the
WebView's loadUrl method cannot be modeled nicely as a widget parameter
(e.g when the widget is rebuilt with the same url value there is no nice
way to tell if you should load that url again or not).
These kind of "edge triggers" fit in the WevView's controller.
initialUrl is completely ignored when the widget is rebuilt.

Adding just these 2 parameters to set up the skeleton for webview
settings and initial parameters.

Settings (like javaScriptMode) are set and updated by rebuilding the
widget.

initialUrl is a covinience parameter for simple use cases, as the
WebView's loadUrl method cannot be modeled nicely as a widget parameter
(e.g when the widget is rebuilt with the same url value there is no nice
way to tell if you should load that url again or not).
These kind of "edge triggers" fit in the WevView's controller.
initialUrl is completely ignored when the widget is rebuilt.
Copy link
Contributor

@bparrishMines bparrishMines left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@amirh amirh merged commit be3e558 into flutter:master Sep 6, 2018
@amirh amirh deleted the webview_3_javascript branch September 6, 2018 15:48
nichtverstehen pushed a commit to nichtverstehen/plugins that referenced this pull request Sep 6, 2018
…r#753)

Adding just these 2 parameters to set up the skeleton for webview
settings and initial parameters.

Settings (like javaScriptMode) are set and updated by rebuilding the
widget.

initialUrl is a covinience parameter for simple use cases, as the
WebView's loadUrl method cannot be modeled nicely as a widget parameter
(e.g when the widget is rebuilt with the same url value there is no nice
way to tell if you should load that url again or not).
These kind of "edge triggers" fit in the WevView's controller.
initialUrl is completely ignored when the widget is rebuilt.
andreidiaconu pushed a commit to andreidiaconu/plugins that referenced this pull request Feb 17, 2019
…r#753)

Adding just these 2 parameters to set up the skeleton for webview
settings and initial parameters.

Settings (like javaScriptMode) are set and updated by rebuilding the
widget.

initialUrl is a covinience parameter for simple use cases, as the
WebView's loadUrl method cannot be modeled nicely as a widget parameter
(e.g when the widget is rebuilt with the same url value there is no nice
way to tell if you should load that url again or not).
These kind of "edge triggers" fit in the WevView's controller.
initialUrl is completely ignored when the widget is rebuilt.
andreidiaconu added a commit to andreidiaconu/plugins that referenced this pull request Feb 17, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
3 participants