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

[webview_flutter] Add an option to set the background color of the webview #3431

Merged

Conversation

e-adrien
Copy link
Contributor

This PR add an option to set a transparent background for the webview.

There are already 2 outdated PR for the same feature :

I took the previous PR made by @crizant and rebase it.

It fixes this issue :

@e-adrien e-adrien requested a review from amirh as a code owner January 15, 2021 11:30
@google-cla google-cla bot added the cla: yes label Jan 15, 2021
@stuartmorgan stuartmorgan added the p: webview_flutter Edits files for a webview_flutter plugin label Jan 29, 2021
@fundup
Copy link

fundup commented Feb 12, 2021

So close to have this feature ... waiting for month as others !!! Hope this merge will be accepted and in the next Webview version !

@crizant
Copy link

crizant commented Mar 10, 2021

There are conflicts, seems @e-adrien you need to rebase again. 🤪

@e-adrien e-adrien force-pushed the webview-transparent-background branch from 50463ad to 073455d Compare March 10, 2021 11:45
@e-adrien
Copy link
Contributor Author

@crizant done 😄 I'll try to rebase it more regularly 😅
Thank you for your original PR by the way.

@davedbase
Copy link

Very excited to have this merged in and available. The overlay/opacity hacks are quite annoying. Also thanks @crizant and @e-adrien for your effort on this.

@e-adrien e-adrien force-pushed the webview-transparent-background branch from 073455d to 4fd8511 Compare March 17, 2021 13:23
@e-adrien e-adrien force-pushed the webview-transparent-background branch from 4fd8511 to a513bb1 Compare March 25, 2021 07:40
@brenoasm
Copy link

Any updates?

@stuartmorgan
Copy link
Contributor

FYI the analyzer issue you are hitting is unrelated to your change; I just landed a fix for it.

stuartmorgan pushed a commit that referenced this pull request Dec 2, 2021
stuartmorgan pushed a commit that referenced this pull request Dec 4, 2021
)

This PR add an option to set the background color of the iOS webview.

Part of #3431
Part of flutter/flutter#29300
stuartmorgan pushed a commit that referenced this pull request Dec 6, 2021
…#4569)

This PR add an option to set the background color of the Android webview.

Part of: #3431
Part of: flutter/flutter#29300
@stuartmorgan
Copy link
Contributor

The individual components have all landed (the Android implementation will auto-publish once the post-submit tests finish), so this can now be reduced down to just the webview_flutter package changes, and a simple test added here, and it should be good to go!

@e-adrien
Copy link
Contributor Author

e-adrien commented Dec 6, 2021

The individual components have all landed (the Android implementation will auto-publish once the post-submit tests finish), so this can now be reduced down to just the webview_flutter package changes, and a simple test added here, and it should be good to go!

Thanks. I will open a PR when the android package will be published.

@stuartmorgan
Copy link
Contributor

There are infrastructure issues that caused #4584 not to land correctly, and are currently preventing the creation of new PRs that land correctly; while we get that resolved I've updated this PR to match #4584 so that we get it landed.

@stuartmorgan stuartmorgan added the waiting for tree to go green (Use "autosubmit") This PR is approved and tested, but waiting for the tree to be green to land. label Dec 7, 2021
@fluttergithubbot fluttergithubbot merged commit 2681f50 into flutter:master Dec 7, 2021
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Dec 7, 2021
amantoux pushed a commit to amantoux/plugins that referenced this pull request Dec 11, 2021
amantoux pushed a commit to amantoux/plugins that referenced this pull request Dec 11, 2021
…utter#4570)

This PR add an option to set the background color of the iOS webview.

Part of flutter#3431
Part of flutter/flutter#29300
amantoux pushed a commit to amantoux/plugins that referenced this pull request Dec 11, 2021
…flutter#4569)

This PR add an option to set the background color of the Android webview.

Part of: flutter#3431
Part of: flutter/flutter#29300
amantoux pushed a commit to amantoux/plugins that referenced this pull request Dec 11, 2021
KyleFin pushed a commit to KyleFin/plugins that referenced this pull request Dec 21, 2021
KyleFin pushed a commit to KyleFin/plugins that referenced this pull request Dec 21, 2021
…utter#4570)

This PR add an option to set the background color of the iOS webview.

Part of flutter#3431
Part of flutter/flutter#29300
KyleFin pushed a commit to KyleFin/plugins that referenced this pull request Dec 21, 2021
…flutter#4569)

This PR add an option to set the background color of the Android webview.

Part of: flutter#3431
Part of: flutter/flutter#29300
KyleFin pushed a commit to KyleFin/plugins that referenced this pull request Dec 21, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cla: yes p: webview_flutter Edits files for a webview_flutter plugin waiting for tree to go green (Use "autosubmit") This PR is approved and tested, but waiting for the tree to be green to land.
Projects
None yet
9 participants