-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Typescript bindings question #1364
Comments
No. Would you open a PR to update the typings? |
Do I just do it against master and then that will suffice for both 6.x and 5.x? (we still need 5.x) |
Yes, this should apply to both 6.x and 5.x. |
Created #1369. It does a bit more than mentioned above, though I believe the additional changes are correct. |
This has been included in the latest patches (6.1.10 and 5.3.10). Thx for the contribution! |
For
MapEvent
, is it possible forpoint: Array<number>,
to instead be e.g.point: [x: number, y: number]
and similarly forlngLat
to be[longitude: Lng, latitude: Lat]
since we know they both just have two elements, or are there reasons against this?EDIT: Also, should
MjolnirEvent
also haveleftButton
andmiddleButton
?The text was updated successfully, but these errors were encountered: