fix markers rerender when click to open InfoWindow

[deadkff01]

Solving issue #269

In Marker.js componentDidUpdate()

this.props.position and prevProps.position will never be equal after component is fully rendered in first time, because these properties are objects, for this reason I use the lat and lng to compare after the first rendering.

[m-bartenev]

@deadkff01 , I came across that issue recently. I did the same comparison but in shouldComponentUpdate method. Can I ask you why you've opted to the componentDidUpdate?
Thanks

[deadkff01]

@m-bartenev, I did the comparison in componentDidUpdate only to keep the same code structure. :)

[m-bartenev]

@deadkff01 , I see :) Thank you for your answer!

[srobertson421]

To anyone who comes across this bug, here is a patch you can use until things are merged, should stop the constant re-renders:

class CustomMarker extends Marker { componentDidUpdate(prevProps) { if( this.props.map !== prevProps.map || this.props.icon.url !== prevProps.icon.url || ( this.props.position.lat !== prevProps.position.lat || this.props.position.lng !== prevProps.position.lng ) ) { if(this.marker) { this.marker.setMap(null); } this.renderMarker(); } } } 

[deadkff01]

Nice @srobertson421

[sscholle]

most unreadable, please rather use a function such as 'arePositionsEqual' to simplify readability. thank you

[deadkff01]

Ok, it's been a long time, I'll see if this change still makes sense, if so, I'll make the change later.
Thanks for the feedback.

[deadkff01]

The problem still persists in the current version of the lib.

I solved your issue in the last commits.