-
Notifications
You must be signed in to change notification settings - Fork 124
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
feat: added set style method on controller #444
base: main
Are you sure you want to change the base?
feat: added set style method on controller #444
Conversation
…hout rebuilding map
# Conflicts: # example/lib/local_style.dart
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.
Hi @itheamc, thank you for keeping your pull request updated with the main branch.
The user has the option to add annotations like circles, symbols and so on. After the user changes the style of the map the annotations added by the user should be still on the map to avoid an inconcistent state between flutter and the maplibre SDKs.
try { | ||
_map.setStyle(styleString); | ||
} on PlatformException catch (e) { | ||
return Future.error(e); | ||
} |
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.
When does _map.setStyle throw a PlatformException? (It's not used in the other methods.)
example/lib/local_style.dart
Outdated
onMapCreated: _onMapCreated, | ||
initialCameraPosition: const CameraPosition(target: LatLng(0.0, 0.0)), | ||
onStyleLoadedCallback: onStyleLoadedCallback, | ||
)); |
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.
Adding the option to change the map to this example could be confusing because this example showcases that the map style can in general be loaded from an local source. Could you create a new page that showcases this functionality please?
Hi @itheamc, I've seen that you clicked on "Resolve conversations" for my comments but 4 haven't been implemented. Could you give some quick details what's the reason please? |
No description provided.