-
Notifications
You must be signed in to change notification settings - Fork 8
feat: Migrates a set of control samples. #1048
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
Conversation
|
Here is the summary of changes. You are about to add 15 region tags.
This comment is generated by snippet-bot.
|
jeantimex
left a comment
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.
Thanks for the great work!
Refactor map library import to simplify code.
Removed empty dependencies section from package.json
Removed empty dependencies section from package.json
| <!-- [START maps_control_simple] --> | ||
| <html> | ||
| <head> | ||
| <title>Adding Controls to the Map</title> |
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.
nit: This title seems a little conflicting with the README, which says this sample shows how to hide a control. Maybe tweak one/both of them?
|
oh wait, Su got to this before me >.< |
This PR makes the following changes: