-
Notifications
You must be signed in to change notification settings - Fork 16
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
Added map-change event for extents #1005
Conversation
ea7bd4a
to
a39cba4
Compare
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.
LGTM! just a few minor suggestions. Great work @yushan-mu!
Also while testing this, discovered a weird bug (#1007), maybe we can take a look at this next |
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.
keyboard interaction test
await context.close(); | ||
}); | ||
|
||
test('Map change event for layers work', async () => { |
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.
test('Map change event for layers work', async () => { | |
test('Map-change event for map-layer dispatched on user interaction', async () => { |
expect(layerClicked).toBe(2); | ||
}); | ||
|
||
test('Map change event for sub-layers work', async () => { |
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.
test('Map change event for sub-layers work', async () => { | |
test('Map-change event for map-extent dispatched on user interaction', async () => { |
… checked/unchecked on the layer menu
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.
Good Work! Squash and merge!
added map-change event for extents, added test cases for map-change events
fixes #907