cybersthang / gmaps-utility-library-dev

Automatically exported from code.google.com/p/gmaps-utility-library-dev
0 stars 0 forks source link

ClusterMarker_ objects ought to be in G_MAP_MARKER_PANE, not G_MAP_MAP_PANE #106

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
What steps will reproduce the problem?
1. Create some markers and pass them to MarkerClusterer.
2. Create GPolylines that run on top of the markers.
3. Zoom out until the aggregate markers appear instead of the markers
themselves.

What is the expected output? What do you see instead?

I expect the GPolylines to be drawn behind the aggregate marker
(ClusterMarker_ object).  Instead, the polylines are drawn on top of the
aggregates, making it difficult to see and click on them.

What version of the product are you using?

1.0.

Please provide any additional information below.

The following line in ClusterMarker_.initialize() is responsible for
putting the aggregate markers onto the bottom-most pane:

map.getPane(G_MAP_MAP_PANE).appendChild(div);

However, I argue that since ClusterMarker_ objects represent (hidden)
GMarkers and indeed are clickable objects, they ought to be put in a higher
plane, such as G_MAP_MARKER_PLANE.  That way, polylines and polygons go
behind both GMarkers and ClusterMarker_s.

Original issue reported on code.google.com by djw...@gmail.com on 26 Apr 2009 at 4:04

GoogleCodeExporter commented 8 years ago
Good point. Assigning to Xiaoxi. Here's the reference for the panes:
http://code.google.com/apis/maps/documentation/reference.html#GMapPane

Original comment by pamela.fox on 26 Apr 2009 at 4:17

GoogleCodeExporter commented 8 years ago
An example as well:
http://djwong.org/.obj/map.php?pagekey=PHOTOS_OREGON

(with patched markerclusterer.js, obviously)

Original comment by djw...@gmail.com on 26 Apr 2009 at 4:42

GoogleCodeExporter commented 8 years ago
Done.

Original comment by frank200...@gmail.com on 26 Apr 2009 at 5:26

GoogleCodeExporter commented 8 years ago
This fix doesnt seem to be uploaded to the trunk?

Original comment by nexflo@gmail.com on 12 Jan 2010 at 7:51