This repository has been archived by the owner on Sep 26, 2022. It is now read-only.
forked from Leaflet/Leaflet
-
Notifications
You must be signed in to change notification settings - Fork 20
Vector layers are cropped when rotated #1
Comments
Can you support for version 0.7.7 |
@hl1994 not possible, too many things related to Leaflet 1.0 which separate layer panes |
Hi @fnicollet. Loving your work so far, but has there been any progress on getting vector layers to rotate correctly? (My Leaflet project has begun using vector layers extensively, so I've had to stop using the rotation branch.) I'd offer to help, but my JS skills probably aren't up to it. |
Hello @jeremygbyrne Fabien |
Hi @fnicollet Thanks for the reply! Does that mean the main branch now supports rotation, somehow? |
Not really, it just means that there is now :
https://github.com/Leaflet/Leaflet/tree/rotate
While it was on my account before:
https://github.com/fnicollet/Leaflet/tree/rotate-master
But both lack updates ...
Fabien
…On Mon, Jan 30, 2017 at 3:47 AM, Jeremy Byrne ***@***.***> wrote:
Hi @fnicollet <https://github.com/fnicollet> Thanks for the reply! Does
that mean the main branch now supports rotation, somehow?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#1 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ABLtoQjyH8xgXJjzUOo4y3ewUqcr3Uxzks5rXU84gaJpZM4JTiY9>
.
--
Fabien Nicollet
Tel : +33 (0)6 60 54 72 20
|
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Following the original issue, there is still one problem with vector rotation, when the map is at zoom 0/1.
We need to backport this fix from @IvanSanchez work Leaflet@a2595bb
Transformation is now done using the center and not the topLeft corner which was clamped (calculated in lat/lng).
The text was updated successfully, but these errors were encountered: