Skip to content
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

Rectangle not supported #16

Closed
axelerate opened this issue Jun 27, 2017 · 5 comments
Closed

Rectangle not supported #16

axelerate opened this issue Jun 27, 2017 · 5 comments
Assignees

Comments

@axelerate
Copy link

Leaflet offers the Rectangle vector shape right out the box, but whenever I attempt to enable transformation and rotate it, the rectangle reverts back to an upright state.
Are there any plans to support rotation of rectangles?

@w8r w8r self-assigned this Jun 27, 2017
@w8r
Copy link
Owner

w8r commented Jun 28, 2017

That's strange, I will look into it

@w8r
Copy link
Owner

w8r commented Jun 29, 2017

Fixed

@w8r w8r closed this as completed Jun 29, 2017
@axelerate
Copy link
Author

axelerate commented Jun 29, 2017

Thanks.
Do note that after this change has been introduced, the following errors pop up when dragging a rectangle, possibly all paths:

Uncaught TypeError: L.DomEvent.fakeStop is not a function
L.Path.Transform-src.js:258 Uncaught TypeError: L.DomEvent.skipped is not a function

@w8r
Copy link
Owner

w8r commented Jun 29, 2017

Please update leaflet to 1.1.0

@dt99jay
Copy link

dt99jay commented Apr 25, 2018

Edit – apologies, please ignore the below, I inadvertently had a version of Mapbox.js included as well as Leaflet.

I'm getting the same error at the end of dragging a rectangle with Leaflet 1.3.1:

L.Path.Transform.js:6
Uncaught TypeError: L.DomEvent.fakeStop is not a function
    at e._onDragEnd (L.Path.Transform.js:6)
    at HTMLDocument.h (leaflet-src.js:4200)
L.Path.Transform.js:6
Uncaught TypeError: L.DomEvent.skipped is not a function
    at e.<anonymous> (L.Path.Transform.js:6)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants