feat: add WidgetLayer
#107
Merged
Conversation
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
Deploying flutter-maplibre with
|
| Latest commit: |
9e9e467
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://9c70032d.flutter-maplibre.pages.dev |
| Branch Preview URL: | https://feat-add-widgetlayer.flutter-maplibre.pages.dev |
josxha
changed the title
WidgetLayerWidgetLayer, turn some controller functions synchonous
josxha
changed the title
WidgetLayer, turn some controller functions synchonous WidgetLayer, make some controller functions sync
josxha
changed the title
WidgetLayer, make some controller functions syncWidgetLayer, add CI for WASM
add in separate pr
Owner
Author
|
The solution is not perfect but good enough for now. The user experience will get some improvement once the merged platform and ui thread get shipped on the stable branch. |
gabbopalma
pushed a commit
to gabbopalma/flutter-maplibre
that referenced
this pull request
Nov 10, 2024
- [x] add WidgetLayer - [x] add example page --- - [x] ~~make toScreenLocation() synchronous~~ - [x] ~~make toLngLat() synchronous~~ - [x] ~~make getVisibleRegion() synchronous~~ - [x] ~~make getMetersPerPixelAtLatitude() synchronous~~ - [x] ~~increase the minimum flutter version to support sync calls~~ ~~This pull request uses the merged raster and ui thread that is currently only available on the flutter main release channel. See flutter/flutter#150525 (comment)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
make toScreenLocation() synchronousmake toLngLat() synchronousmake getVisibleRegion() synchronousmake getMetersPerPixelAtLatitude() synchronousincrease the minimum flutter version to support sync callsThis pull request uses the merged platform and ui thread that is currently only available on the flutter main release channel. See flutter/flutter#150525 (comment)