-
Notifications
You must be signed in to change notification settings - Fork 3.6k
[google_adsense] (Experimental) AdSense plugin for Flutter Web #6871
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
Changes from 1 commit
Commits
Show all changes
113 commits
Select commit
Hold shift + click to select a range
ddb2de6
Initial version of AdSense plugin for Flutter Web
sokoloff06 968dcdb
Merge branch 'flutter:main' into main
sokoloff06 3c99bae
Merge branch 'flutter:main' into main
sokoloff06 c37d644
WIP: apply flutter_packages analysis_options.yaml and fixing warnings
sokoloff06 dfae8ee
Merge remote-tracking branch 'origin/main'
sokoloff06 15826c0
remove local analysis_options.yaml
sokoloff06 6050087
Update README.md, pubspec.yaml, work on example and linter warnings
sokoloff06 25158d3
Update public API, add example, update README.md, add more docs and s…
sokoloff06 91ae43f
Update public API, add example, update README.md, add more docs and s…
sokoloff06 4308c93
Merge remote-tracking branch 'origin/main'
sokoloff06 52fb128
Merge remote-tracking branch 'origin/main'
sokoloff06 a562233
Merge remote-tracking branch 'origin/main'
sokoloff06 500483d
Fix analyzer warnings
sokoloff06 65c61ab
Updated README.md
sokoloff06 065e6fe
Add documentation, copyright, initial tests, update README.md
sokoloff06 b99f4f7
Fix linter error
sokoloff06 2c7db93
Make plugin closer to publishable state: update copyright, commit mis…
sokoloff06 1697032
Rename adView to adUnit, fix deps and update README.md
sokoloff06 875f621
Merge branch 'main' into main
sokoloff06 5773e5d
Refactoring + unit tests
sokoloff06 537d611
Merge remote-tracking branch 'origin/main'
sokoloff06 f105166
Refactoring + unit tests
sokoloff06 7d3b9c2
Upgrade dart SDK
sokoloff06 de1fe7c
Upgrade package:web and dart
sokoloff06 458cb4e
Add integration test
sokoloff06 ffab481
Clarify 403 error in README.md
sokoloff06 257007f
Rename ad_unit_widget.dart and solve dart analysis errors
sokoloff06 7701a7c
Merge branch 'main' into main
sokoloff06 172dd00
Add styling property for adUnit CSS customization
sokoloff06 c427a8a
Merge remote-tracking branch 'origin/main'
sokoloff06 689b1a6
Merge branch 'main' into main
sokoloff06 215da90
Code format
sokoloff06 3232f0e
Fix repo rules
sokoloff06 d59f7b7
Fix repo rules
sokoloff06 98b0845
Update min Dart version and readme links
sokoloff06 0232565
Update min SDK versions for dependencies compatibility
sokoloff06 f2b9c06
Fix typo
sokoloff06 db13a98
Revert README.md
sokoloff06 51e2042
Update README.md
sokoloff06 c57e03f
Merge branch 'main' into main
sokoloff06 e6842f2
Update README.md, rename class Adsense into AdSense
sokoloff06 913f347
Update packages/google_adsense/example/pubspec.yaml
sokoloff06 9ba0ba5
Use instance property to access AdSense singleton instance
sokoloff06 eb20328
Merge remote-tracking branch 'sokoloff06/main' into fork/sokoloff06/main
sokoloff06 2a8a998
Add explanation about local DNS records for testing environment
sokoloff06 1c29983
Use strong String type for additionalParams argument values
sokoloff06 d07ea36
Remove as JS* casts, change isAdTest default value, improve some docs…
sokoloff06 d48aa16
Remove as JS* casts, change isAdTest default value, improve some docs…
sokoloff06 094460b
Add screenshots to pubspec.yaml
sokoloff06 3f58513
Simplify ResizeObserver, define adsbygoogle JS variable
sokoloff06 988d3f7
Code cleanup
sokoloff06 6c244fd
Merge tests
sokoloff06 410a7eb
Tests cleanup
sokoloff06 dc78f2e
Provide adSense global variable for convenience
sokoloff06 12e657d
Remove adClient override logic,
sokoloff06 3a820bc
Update adUnit API with AdUnitConfiguration and named constructors for…
sokoloff06 1d288db
Remove test_ad.html, format code
sokoloff06 d214fc0
Merge branch 'main' into main
sokoloff06 3fda3da
Ignore avoid_print in test, remove redundant arguments
sokoloff06 0258b86
use testWidgets in integration tests
sokoloff06 b9f61b8
Fix adUnit returns Widget type test
sokoloff06 c509169
Update README.md and add LICENSE
sokoloff06 4096469
Tweak JS-interop.
ditman fbde1c2
Tweak ad unit param enums.
ditman bbaa9db
Use new js-interop init. Remove array fallback from addMasterScript.
ditman 9742dcf
Use new types in ad_unit_configuration
ditman 3acd43c
Use Size for ad unit widget. Some tweaks.
ditman bc140f4
dart format .
ditman bd7f839
Fix HTMLDivElement cast
ditman 604b87b
Use offsetWidth/height
ditman 4175da6
Fix tests
ditman 36d2550
Merge pull request #1 from ditman/adsense-js-interop
sokoloff06 875f72c
Remove width and height css rules, fix test app
sokoloff06 41edb56
TO_REVIEW: Optimize? - Reuse single static ResizeObserver as per [thi…
sokoloff06 39dfc97
Hide logs from release, add AdStatus enum,
sokoloff06 1bbaeee
Fix AdStatus
sokoloff06 7a31b8e
Fix unit tests CI
sokoloff06 aeab558
Add TrustedTypes support
sokoloff06 3235d27
Remove stub tests
sokoloff06 5a7abad
Fix missing type annotation
sokoloff06 9dca5d1
Update README.md excerpts
sokoloff06 7d828cf
Update example to more complex layout
sokoloff06 d9dbc79
Merge branch 'main' into main
sokoloff06 308a325
Simplify API: remove cssText and add docs to use Flutter instruments,…
sokoloff06 baac6ba
Tweak initialization. Support prior initialization in HTML.
ditman 2d9e5bd
Add integration tests mocking window.adsbygoogle. Tweaks.
ditman 456da08
Ensure package compiles in Wasm.
ditman 5f2b324
Tweak the API of AdSense. Rename internal adClient to adClientId. Mak…
ditman ffd8c4a
Remove adClient from AdUnitParams object, because it's always passed …
ditman 7e4ddc9
Pass clientId when creating ad units from AdSense instance.
ditman d2b42f1
Simplify some more code.
ditman e5da3bf
Make isAdTest a normal member of the config. Remove unused getters.
ditman 9ce3347
Remove AdUnitWidget._internal constructor.
ditman 8bda3fd
Make as much API private as possible.
ditman 500337e
Use new constructor instead of the removed factory.
ditman 06886dd
Allow limiting ads by height, update example
sokoloff06 a36f9f8
Allow limiting ads by height, update example
sokoloff06 7f55807
Merge pull request #2 from ditman/add-mock-adsense-tests
sokoloff06 ba38273
Fix CI analyzer fails
sokoloff06 1914f6b
Fix CI analyzer fails
sokoloff06 c5e46a6
Fix CI analyzer fails
sokoloff06 3bb56b9
Make sure <ins> initial size is equal to wrapping div
sokoloff06 d7abe89
dart format
sokoloff06 a92864f
Add docs on using BoxConstraints
sokoloff06 4e52e92
Update tests to match AdSense behavior, and our code.
ditman 44fc9e6
Update README.md and comments
sokoloff06 49a7a75
Merge pull request #3 from ditman/fix-ins-in-integration-test
sokoloff06 683fe65
Update README.md
sokoloff06 06803b8
Update README.md
sokoloff06 c0d9ec3
Merge remote-tracking branch 'sokoloff06/main' into fork/sokoloff06/main
sokoloff06 6dbecd7
Merge branch 'main' into main
sokoloff06 53f47e4
Move exports to experimental folder, update README.md
sokoloff06 9735ba0
Fix CI
sokoloff06 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Update README.md, pubspec.yaml, work on example and linter warnings
- Loading branch information
commit 605008758c4a051970fe47185e88e684decb3979
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,3 +1,3 @@ | ||
| ## 0.0.1 | ||
| ## 0.1.0 | ||
|
|
||
| * Initial MVP release. | ||
| * Initial release. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,39 +1,32 @@ | ||
| Plugin allowing Adsense integration with Flutter Web | ||
|
|
||
| [//]: # (## Features) | ||
|
|
||
| [//]: # () | ||
| [//]: # (TODO: List what your package can do. Maybe include images, gifs, or videos.) | ||
|
|
||
| [//]: # () | ||
| [//]: # (## Getting started) | ||
|
|
||
| [//]: # () | ||
| [//]: # (TODO: List prerequisites and provide or point to information on how to) | ||
|
|
||
| [//]: # (start using the package.) | ||
|
|
||
| [//]: # () | ||
| [//]: # (## Usage) | ||
|
|
||
| [//]: # () | ||
| [//]: # (TODO: Include short and useful examples for package users. Add longer examples) | ||
|
|
||
| [//]: # (to `/example` folder.) | ||
|
|
||
| [//]: # () | ||
| [//]: # (```dart) | ||
|
|
||
| [//]: # (const like = 'sample';) | ||
|
|
||
| [//]: # (```) | ||
|
|
||
| [//]: # () | ||
| [//]: # (## Additional information) | ||
|
|
||
| [//]: # () | ||
| [//]: # (TODO: Tell users more about the package: where to find more information, how to) | ||
|
|
||
| [//]: # (contribute to the package, how to file issues, what response they can expect) | ||
|
|
||
| [//]: # (from the package authors, and more.) | ||
| # google_adsense | ||
| [Google AdSense](https://adsense.google.com/intl/en_us/start/) plugin for Flutter Web | ||
|
|
||
| Generally integration with AdSense requires: | ||
|
|
||
| 1. Adding an [AdSense code](https://support.google.com/adsense/answer/9274634?hl=en&ref_topic=28893&sjid=9002959242386336076-EU) usually in between the `<head></head>` tag to connect your site with AdSense account. Allows using **Auto ads** | ||
| 2. Adding [ad unit code](https://support.google.com/adsense/answer/9274025?sjid=9002959242386336076-EU) in between the `<body><body/>` tags of your pages where you want ad to appear. Ad unit code snippet is unique per **ad unit** that you need to create in your AdSense account first. You might have several ad units added to your website | ||
|
|
||
| Since Flutter apps are not rendered as traditional HTML pages and there is no easy way to access final DOM tree from Dart, this plugin provides a Widget (`AdViewWidget`) that you can configure as an ad unit and place in the desired location in your app UI | ||
|
|
||
|
|
||
| ## Installation | ||
| run `flutter pub add google_adsense` | ||
|
|
||
| ## Usage | ||
| #### Initialize AdSense | ||
| Before displaying ads, initialize the AdSense with your ad client ID. | ||
sokoloff06 marked this conversation as resolved.
Outdated
Show resolved
Hide resolved
|
||
| ```dart | ||
| import 'package:google_adsense/adsense.dart'; | ||
|
|
||
| void main() { | ||
| Adsense().initialize('your-ad-client-id'); | ||
| } | ||
| ``` | ||
| #### Display AdViewWidget | ||
| ```dart | ||
| import 'package:google_adsense/adsense.dart'; | ||
| // ... | ||
| val adWidget = Adsense().adView( | ||
| adSlot: 'your-ad-slot-id', | ||
| ); | ||
| ``` | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,3 +1 @@ | ||
| library adsense_web_standalone; | ||
|
|
||
| export 'src/adsense_stub.dart' if (dart.library.html) 'src/adsense_web.dart'; |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,58 +1,26 @@ | ||
| name: google_adsense | ||
| description: Google Adsense plugin for Flutter Web | ||
| repository: https://github.com/flutter/packages/tree/main/packages/google_adsense | ||
| issue_tracker: https://github.com/flutter/flutter/issues?q=is%3Aissue+is%3Aopen+label%3A%22p%3A+google_adsense%22 | ||
| version: 0.0.1 | ||
| homepage: | ||
|
|
||
| environment: | ||
| sdk: '>=3.1.0 <4.0.0' | ||
| flutter: ">=1.17.0" | ||
| sdk: ^3.1.0 | ||
| flutter: ^1.17.0 | ||
|
|
||
| dependencies: | ||
| flutter: | ||
| sdk: flutter | ||
| web: ^0.5.1 | ||
|
|
||
| dev_dependencies: | ||
| build_runner: ^2.4.9 | ||
| flutter_lints: ^2.0.0 | ||
| flutter_test: | ||
| sdk: flutter | ||
| flutter_lints: ^2.0.0 | ||
| test: ^1.24.9 | ||
| mockito: ^5.4.4 | ||
| build_runner: ^2.4.9 | ||
|
|
||
| # For information on the generic Dart part of this file, see the | ||
| # following page: https://dart.dev/tools/pub/pubspec | ||
|
|
||
| # The following section is specific to Flutter packages. | ||
| flutter: | ||
|
|
||
| # To add assets to your package, add an assets section, like this: | ||
| # assets: | ||
| # - images/a_dot_burr.jpeg | ||
| # - images/a_dot_ham.jpeg | ||
| # | ||
| # For details regarding assets in packages, see | ||
| # https://flutter.dev/assets-and-images/#from-packages | ||
| # | ||
| # An image asset can refer to one or more resolution-specific "variants", see | ||
| # https://flutter.dev/assets-and-images/#resolution-aware | ||
| test: ^1.24.9 | ||
|
|
||
| # To add custom fonts to your package, add a fonts section here, | ||
| # in this "flutter" section. Each entry in this list should have a | ||
| # "family" key with the font family name, and a "fonts" key with a | ||
| # list giving the asset and other descriptors for the font. For | ||
| # example: | ||
| # fonts: | ||
| # - family: Schyler | ||
| # fonts: | ||
| # - asset: fonts/Schyler-Regular.ttf | ||
| # - asset: fonts/Schyler-Italic.ttf | ||
| # style: italic | ||
| # - family: Trajan Pro | ||
| # fonts: | ||
| # - asset: fonts/TrajanPro.ttf | ||
| # - asset: fonts/TrajanPro_Bold.ttf | ||
| # weight: 700 | ||
| # | ||
| # For details regarding fonts in packages, see | ||
| # https://flutter.dev/custom-fonts/#from-packages | ||
| topics: | ||
| - monetization | ||
| - ads |
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.