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

CB-9393: Documenting how to embed a SystemWebView #544

Closed
wants to merge 1 commit into from

Conversation

infil00p
Copy link
Member

@infil00p infil00p commented Mar 8, 2016

Redo of the pull request to update the SystemWebView documentation.

@infil00p
Copy link
Member Author

@riknoll Bump, I've updated the branch

parser.parse(this);

SystemWebView webView = (SystemWebView) findViewById(R.id.tutorialView);
webInterface = new CordovaWebViewImpl(new SystemWebViewEngine(webView));
Copy link
Contributor

Choose a reason for hiding this comment

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

webInterfaceneeds to be declared

@riknoll
Copy link
Contributor

riknoll commented Mar 15, 2016

Minor comments, but otherwise looks pretty good!

@infil00p
Copy link
Member Author

Blargh! I need to hunt down the merge conflict before re-sending this pull request. I have no idea what changed on this file, though.

@infil00p infil00p closed this Mar 22, 2016
following:
private CordovaInterfaceImpl iface = new CordovaInterfaceImpl(this);

1. Add the following to intialize the WebView

Choose a reason for hiding this comment

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

initialize*

Choose a reason for hiding this comment

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

what if i want to override

setWebChromeClient(WebChromeClient client)

method of systemWebView

I got rutime error of class cast exception

Copy link
Member

Choose a reason for hiding this comment

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

Commenting on old docs PRs is never the right solution. Better create a Stack Overflow question.

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

5 participants