Doesn't seem like CI is related to my PR...

^^ I stand corrected

Really?
653:63 error Trailing spaces not allowed no-trailing-spaces

LOL when i looked at it that message hadn't loaded yet and I got super confused. Will update!!

Having onBlabblaChange calback in interface I always try to find blabbla prop on Component public interface.
Here we have onMapTypeIdChange but mapTypeId is not present in props, for me it's confusing a little.
Not sure what to do, or to add mapTypeId as a controllable property, or to somehow change the event name. IMO first is overhead, and I can't suggest a good name for the second.
Your opinion?

That's a great point, I would say considering it's something that a user might want to set.. for example "oh, i want to show a satellite map here" we probably want it to be controlled. This seems reasonable to me.
https://developers.google.com/maps/documentation/javascript/reference#MapTypeId. The real truth though, is isn't it wrapped up in the options object? Like a lot of the things you can do with this library are hidden behind that sort of big options object that you can fill with styles, or whatever you want, so in a sense it's kinda handled. Thoughts?

Please add 1-2 lines about in documentation, and let's merge then.

Let's think about after merge, as now I have no ideas ;-)

google-map-react@0.21.4

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.