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

Zoom and then the circles disappear #106

Closed
dianaclem opened this issue Sep 2, 2015 · 8 comments
Closed

Zoom and then the circles disappear #106

dianaclem opened this issue Sep 2, 2015 · 8 comments
Assignees
Labels
Milestone

Comments

@dianaclem
Copy link
Contributor

dianaclem commented Sep 2, 2015

Ran O'Hare and Midway on Samsung (both Firefox and Google). It works fine. I zoom in with my fingers and the polygon stays but the circles/points disappear.

screenshot_2015-09-02-09-28-00 missing dots

@tomschenkjr
Copy link
Contributor

Is this still a thing?

@dianaclem
Copy link
Contributor Author

Yes, inconsistently. Seems like a Leaflet bug, but we haven’t had time to look too far into it.

From: Tom Schenk Jr [mailto:notifications@github.com]
Sent: Wednesday, November 18, 2015 11:56 PM
To: Chicago/opengrid opengrid@noreply.github.com
Cc: Diana Clem dclem@eki-consulting.com
Subject: Re: [opengrid] Zoom and then the circles disappear (#106)

Is this still a thing?


Reply to this email directly or view it on GitHubhttps://github.com//issues/106#issuecomment-157959740.

@tomschenkjr
Copy link
Contributor

No problem, just doing a review/clean-up of tasks.

From: dianaclem [mailto:notifications@github.com]
Sent: Thursday, November 19, 2015 9:24 AM
To: Chicago/opengrid opengrid@noreply.github.com
Cc: Schenk, Tom Tom.Schenk@cityofchicago.org
Subject: Re: [opengrid] Zoom and then the circles disappear (#106)

Yes, inconsistently. Seems like a Leaflet bug, but we haven’t had time to look too far into it.

From: Tom Schenk Jr [mailto:notifications@github.com]
Sent: Wednesday, November 18, 2015 11:56 PM
To: Chicago/opengrid <opengrid@noreply.github.commailto:opengrid@noreply.github.com>
Cc: Diana Clem <dclem@eki-consulting.commailto:dclem@eki-consulting.com>
Subject: Re: [opengrid] Zoom and then the circles disappear (#106)

Is this still a thing?


Reply to this email directly or view it on GitHubhttps://github.com//issues/106#issuecomment-157959740.


Reply to this email directly or view it on GitHubhttps://github.com//issues/106#issuecomment-158088390.


This e-mail, and any attachments thereto, is intended only for use by the addressee(s) named herein and may contain legally privileged and/or confidential information. If you are not the intended recipient of this e-mail (or the person responsible for delivering this document to the intended recipient), you are hereby notified that any dissemination, distribution, printing or copying of this e-mail, and any attachment thereto, is strictly prohibited. If you have received this e-mail in error, please respond to the individual sending the message, and permanently delete the original and any copy of any e-mail and printout thereof.

@PriyaDoIT
Copy link

see if a leaflet upgrade is available and can clear this issue up.

@PriyaDoIT PriyaDoIT removed the Low label Aug 19, 2016
@PriyaDoIT PriyaDoIT added this to the v1.3.0 milestone Aug 19, 2016
@PriyaDoIT
Copy link

see #186 for another example of this

Hi @rladines has the leaflet already been upgraded? I can't recreate this issue and I've tried testing this with a number of datasets now.

@reve0716 reve0716 added in progress and removed ready labels Sep 9, 2016
@PriyaDoIT PriyaDoIT modified the milestones: v1.3.0, v1.4.0 Oct 26, 2016
@rladines rladines assigned reve0716 and PriyaDoIT and unassigned rladines Nov 14, 2016
@rladines
Copy link
Collaborator

@PriyaDoIT and @reve0716 pls. confirm that this is no longer occurring with Leaflet 1.0.1. thanks

@PriyaDoIT
Copy link

I was not able to replicate the issue on INT.

@rladines
Copy link
Collaborator

Closing this since the missing-dots issue has not re-surfaced with the upgraded Leaflet lib.

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

No branches or pull requests

5 participants