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

GL and canvas features don't redraw when their draw methods are called. #585

Merged
merged 2 commits into from
Jun 9, 2016

Conversation

manthey
Copy link
Contributor

@manthey manthey commented Jun 6, 2016

Rather than calling draw on a layer or feature, you have to call draw on the whole map. This fixes the issue. Resolves issue #470.

Fix the hurricanes example's line highlight.

Fix the picking example (though keep it disabled).

Rather than calling draw on a layer or feature, you have to call draw on the whole map.  This fixes the issue.  Resolves issue #470.

Fix the hurricanes example's line highlight.

Fix the picking example (though keep it disabled).
@codecov-io
Copy link

codecov-io commented Jun 6, 2016

Current coverage is 74.03%

Merging #585 into master will increase coverage by <.01%

@@             master       #585   diff @@
==========================================
  Files            83         85     +2   
  Lines          7320       7375    +55   
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
+ Hits           5419       5460    +41   
- Misses         1901       1915    +14   
  Partials          0          0          

Powered by Codecov. Last updated by 7d1f16a...cfb3df4

@aashish24
Copy link
Member

thanks @manthey, i will have a closer look for review.

@aashish24
Copy link
Member

@manthey LGTM 👍

@manthey manthey merged commit 4b00f71 into master Jun 9, 2016
@manthey manthey deleted the gl-features-draw branch June 9, 2016 17:33
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

Successfully merging this pull request may close these issues.

3 participants