google-map icon indicating copy to clipboard operation
google-map copied to clipboard

Fix for marker content update bug

Open tst-dhudlow opened this issue 9 years ago • 9 comments

See #286

tst-dhudlow avatar May 03 '16 11:05 tst-dhudlow

Please add a test.

ebidel avatar May 03 '16 15:05 ebidel

ping

ebidel avatar Jun 06 '16 01:06 ebidel

When I run the current tests with polymer test I get a bunch of failures. (Timeouts and expected { Object (__gm, gm_accessors_, ...) } to equal undefined...). Can you point me to any docs on how to get these running so I can add?

tst-dhudlow avatar Jun 15 '16 12:06 tst-dhudlow

Running them myself, it looks like there are errors. I can help get that working again.

Mind just adding a test for yourself?

ebidel avatar Jun 15 '16 19:06 ebidel

Sure thing, I'll give it a shot. I just figured I was probably not configured correctly.

tst-dhudlow avatar Jun 20 '16 15:06 tst-dhudlow

FYI, still working on reproducing this during tests. Having a bit of a time with it, but making progress.

tst-dhudlow avatar Jun 29 '16 00:06 tst-dhudlow

Test created.

Note that this test will also pass with only changes to google-map-marker.html from #294

I still believe this change is logically correct though it may not technically be necessary after we hammer out #294.

tst-dhudlow avatar Jun 29 '16 19:06 tst-dhudlow

TBH, I'm not sure about this change. Generally, hacking around timing issues like this means there's an underlying problem that needs to be addressed. In this case, #286 is using google-map-marker outside of google-map. That's not intended behavior.

ebidel avatar Jul 05 '16 13:07 ebidel

Well, it's possible you're right that issues related to #294 are the real problem. I could try to verify this by tweaking the test, but it's my understanding that the issues is caused because when the marker is detached, its content is updated, and then it's re-attached, it doesn't update its content. Templates detach stuff from the DOM so they can be re-used and that's why this came up here.

The question is, do you want content to be updated if it is added/modified while the marker is in a detached state?

tst-dhudlow avatar Jul 05 '16 17:07 tst-dhudlow