-
-
Notifications
You must be signed in to change notification settings - Fork 860
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
Support Flutter 3.10 (& Rework Tile Providers) #1512
Conversation
Reworked tile provider implementations to improve performance and meet guidelines Removed 'Delete Old Runs' workflow to prevent deletion of built artifacts
Updated example app's 'build.gradle'
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just some very minor nitpick, otherwise LGTM 👍
lib/src/layer/tile_layer/tile_provider/file_providers/tile_provider_html.dart
Outdated
Show resolved
Hide resolved
lib/src/layer/tile_layer/tile_provider/network_image_provider.dart
Outdated
Show resolved
Hide resolved
Removed fallback to `NetworkTileProvider` when `FileTileProvider` used on web
Checks won't be 100% successful: see axel-op/dart-package-analyzer#23. However, if everything is building, it can be bypassed. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
#1512 (5fb3b3b) bumped the minimum Flutter SDK to 3.10, but did not change the minimum Dart SDK. This PR changes the Dart SDK to 3.0 minimum, enabling the new features and performance benefits. In addition, this PR: * Removes the Tuple dependency in favour of Records * Uses new pattern matching switches in some locations * Seals the `TileDisplay` class to benefit from new pattern matching switches, instead of switching on `runtimeType` * Deprecates `TileUpdateTransformers.alwaysLoadAndPrune`, as it should be unnecessary compared to `ignoreTapEvents` * Improves some documentation * Includes some other more minor improvements/changes This PR also fixes the failing Android builds by upgrading Gradle.
dart fix
HttpOverrides
/zoning strategiesNetworkTileProvider
whenFileTileProvider
used on webIncludes breaking changes.