Skip to content
This repository has been archived by the owner on Nov 30, 2018. It is now read-only.

Markers doRebuildAll=false still rebuilds all markers #1485

Closed
rcino opened this issue Sep 7, 2015 · 1 comment · Fixed by #1799
Closed

Markers doRebuildAll=false still rebuilds all markers #1485

rcino opened this issue Sep 7, 2015 · 1 comment · Fixed by #1799
Assignees
Milestone

Comments

@rcino
Copy link
Contributor

rcino commented Sep 7, 2015

Not sure if this is related to #1447

When pushing markers to array, even with doRebuildAll=false specified, markers seem to be rebuilt (assuming since they flash). When adding 1000's of markers, this gets cumbersome.

This was working with 2.0.12, but seems to no longer work with 2.2.0. I created a Plunker to compare. You can simply comment out the 2.2.0 scripts and uncomment the 2.0.12 scripts in the HTML to see the difference. I added Animation to markers so it's easier to see. Click "Add Marker" to randomly add a marker and see them re-build.

Plunker example

Love this library, thanks in advance for the help. Happy to add to the example if needed.

@nmccready nmccready modified the milestones: 2.2.1, 2.2.2 Sep 8, 2015
@nmccready nmccready modified the milestones: 2.2.2, 2.3.1 Jan 25, 2016
@nmccready nmccready modified the milestones: 2.3.1, 2.3.2 Feb 1, 2016
rcino added a commit to rcino/angular-google-maps that referenced this issue Apr 7, 2016
Change model-key to use lodash to compare objects
 - When objects (outside of coords) are compared via ===, the comparison always returns falls, even if objects are same.  Thus, with a false comparison, all markers get marked as needing update, negating doRebuildAll=false.  This fixes false negative on comparison.
 - Add simple examples page with examples controller for issue angular-ui#1485.
 - Update model-key spec to include a non-coord comparison on markers

 Closes angular-ui#1485
@nmccready nmccready self-assigned this Jun 8, 2016
@pluthong
Copy link

is it fixed?

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

Successfully merging a pull request may close this issue.

3 participants