Skip to content

Add platform view support to flutter_tizen #462

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

Merged
merged 12 commits into from
Nov 16, 2022

Conversation

bbrto21
Copy link
Contributor

@bbrto21 bbrto21 commented Nov 14, 2022

  • This is from the webview_flutter plugin.
  • Mimic original flutter lib's directory structure.
  • Keep syncing platform_view to latest in flutter

Signed-off-by: Boram Bae [email protected]

* This is from the webview_flutter plugin.
* Mimic original flutter lib's directory structure.

Signed-off-by: Boram Bae <[email protected]>
@bbrto21
Copy link
Contributor Author

bbrto21 commented Nov 14, 2022

How about this PR? I just moved this from webview_flutter to flutter_tizen package.
If this is merged, webview_flutter will depend on it.

Copy link
Member

@bwikbs bwikbs left a comment

Choose a reason for hiding this comment

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

Wow! It's the exactly same what I wanted! 👍

@bbrto21
Copy link
Contributor Author

bbrto21 commented Nov 14, 2022

Wow! It's the exactly same what I wanted! +1

Thanks to @swift-kim

@swift-kim
Copy link
Member

Please provide something like flutter-tizen/plugins#54 (review) so that we can track the original source code and make updates later. I think now is the good chance to sync the latest source code from the framework but not just copy the old webview_flutter's code.

* Keep syncing this to bin/internal/flutter.version after this patch.

Signed-off-by: Boram Bae <[email protected]>
@bbrto21
Copy link
Contributor Author

bbrto21 commented Nov 14, 2022

@swift-kim I added sources and update implementations to the latest.

@bbrto21 bbrto21 requested a review from swift-kim November 15, 2022 06:19
Signed-off-by: Boram Bae <[email protected]>
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.

// ignore_for_file: public_member_api_docs
Copy link
Member

Choose a reason for hiding this comment

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

This might be removed in the future.

Signed-off-by: Boram Bae <[email protected]>
Signed-off-by: Boram Bae <[email protected]>
Signed-off-by: Boram Bae <[email protected]>
Signed-off-by: Boram Bae <[email protected]>
Signed-off-by: Boram Bae <[email protected]>
Signed-off-by: Boram Bae <[email protected]>
@bbrto21 bbrto21 changed the title Add platform_view to flutter_tizen Add platform view support to flutter_tizen Nov 16, 2022
@bbrto21 bbrto21 merged commit da82a70 into flutter-tizen:master Nov 16, 2022
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.

3 participants