on-click attribute for map markers about google-map

[ebidel]

I don't think the issue title matches what the PR is doing, but I'm not sure we want to add all these listeners to every instance of a marker element. Most people will never listen to them so it's wasteful.

It's fairly easy to add manually:

google.maps.event.addListener(googleMarkerEl.marker, 'click', function(e) { })

[positlabs]

What if we add the event listeners when the attribute is changed (e.g. "on-clickChanged": function(){add listener}) as opposed to adding them all during initialization?

[ebidel]

Polymer treats on-* attributes special for the declarative event handlers. Perhaps this is a opt-in boolean attribute someone can use on the element? <google-map-marker events> would mean setup the listeners on init.

[positlabs]

Ohh yes I like that approach. @DracoNoir, do you want to add this as part of your PR?

[DracoNoir]

Yes, although I'm going to go a step farther and separate it into two sets of events: clickEvents and mouseEvents. This will allow a developer to have finer-grained control over which listeners are registered (I suspect that the former will be used more than the latter).

[ebidel]

Should be fixed in https://github.com/GoogleWebComponents/google-map/pull/102/files