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.

Sign up for GitHub

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

Jump to bottom

Snapshot tool includes an orange square if leaflet map is used #1325

Closed
Gnafu opened this issue Dec 6, 2016 · 0 comments
Closed

Snapshot tool includes an orange square if leaflet map is used #1325

Gnafu opened this issue Dec 6, 2016 · 0 comments
Assignees
@offtherailz
Labels
bug LeafLetJS

Comments

@Gnafu
Copy link
Contributor

Gnafu commented Dec 6, 2016

See below some snapshots: the orange square should not be in the snapshot.
OpenLayers map is not affected by this bug.

snapshot
snapshot_2
offtherailz added a commit to offtherailz/MapStore2 that referenced this issue Dec 6, 2016
@offtherailz
Fix geosolutions-it#1325. Get only main map svg
c9f6493
offtherailz added a commit to offtherailz/MapStore2 that referenced this issue Dec 6, 2016
@offtherailz
Fix geosolutions-it#1325. Get only main map svg
47d1a7c
@offtherailz offtherailz self-assigned this Dec 6, 2016
@ghost ghost removed the pending review label Dec 6, 2016
mbarto pushed a commit that referenced this issue Dec 13, 2016
@offtherailz @mbarto
Snapshot marker support (#1328)
5400892 
* Fix #1325. Get only main map svg

* Introduced Support for marker snapshot in leaflet

There is still a problem when the map is panned.
When possible, you have to parse the transform inline style of the mapdiv to properly shift the clonded marker div to snap (or properly set the canvas).

* fixed issues with query selector and FF-31.0.0
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@offtherailz offtherailz

Labels
bug LeafLetJS
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@offtherailz @Gnafu