Zn2O / gwt-google-apis

Automatically exported from code.google.com/p/gwt-google-apis
0 stars 0 forks source link

InfoWindow should be a subclass of Overlay so that event.getOverlay() will return them #170

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Found in Release: r290

Detailed description:
This code should assign an InfoWindow to "overlay", if a MapClickEvent occurs 
on top of it:

map.addMapClickHandler(new MapClickHandler() { 
                        public void onClick(MapClickEvent event) { 
                              Overlay overlay = event.getOverlay()
                        } 
                }); 

Eric says:
GInfoWindow is supposed to be an Overlay and returned from 
event.getOverlay(), but don't count on it working because gwt-maps doesn't 
implement InfoWindow as a subclass of Overlay. 

Links to the relevant GWT Developer Forum posts:

http://groups.google.com/group/Google-Web-
Toolkit/browse_thread/thread/bc38a0a70c09e551

Original issue reported on code.google.com by nevin.fr...@gmail.com on 10 Sep 2008 at 10:04

GoogleCodeExporter commented 9 years ago

Original comment by galgwt.reviews@gmail.com on 10 Sep 2008 at 10:35

GoogleCodeExporter commented 9 years ago

Original comment by mmendez+personal@google.com on 15 Sep 2008 at 1:33

GoogleCodeExporter commented 9 years ago
Patch attached.  It makes InfoWindow extend the ConcreteOverlay class.
It also adds the methods isInfoWindow(), isPolyline(), isPolygon(), and 
isMarker() to
the overlay class.

Original comment by galgwt.reviews@gmail.com on 16 Sep 2008 at 6:17

Attachments:

GoogleCodeExporter commented 9 years ago
Committed as r802.

Original comment by galgwt.reviews@gmail.com on 18 Sep 2008 at 8:27

GoogleCodeExporter commented 9 years ago

Original comment by galgwt.reviews@gmail.com on 13 Oct 2008 at 8:54

GoogleCodeExporter commented 9 years ago

Original comment by galgwt.reviews@gmail.com on 11 Dec 2008 at 3:08

GoogleCodeExporter commented 9 years ago

Original comment by zundel@google.com on 21 Dec 2009 at 1:57