I'm not sure how to go about this to keep it fully flexible and useful. The first step is just to let people change the icon of all the KML placemarks to the same thing (not too difficult). To be truly useful though, it should be possible to distinguish different types of content - perhaps allowing people to choose an icon per node type. On the other hand, I also think it may be useful to have all the icons within one feed (eg tagged with a specific term, from a group, from search results) have the same icon so as to distinguish the different feeds.

This topic needs some more thought still, but I'd probably implement these in that order.

CommentFileSizeAuthor
#6 custom_markers.diff3.98 KBmarcoBauli
#3 kml.patch3.04 KBmarcoBauli

Comments

geodaniel’s picture

geodaniel commented
Category: task » feature
marcoBauli’s picture

marcoBauli commented

totally second this, +1 tester ;)

marcoBauli’s picture

marcoBauli commented
Status: Active » Needs review
StatusFileSize
new kml.patch3.04 KB

the patch attached allows to customize the markers with a custom one, plus another second custom marker for rollover. Please review, thx

raintonr’s picture

raintonr commented

+1 for the 'icon per content type' idea here :)

raintonr’s picture

raintonr commented

It took a small change to get the above patch working with latest KML.module we had but now the system is functioning perfectly, thanks.

Taking this one step further as per my previous comment have created a related feature request; "Some KML options should be per content type" (http://drupal.org/node/181048). I've added that as the per-content type icon change I have made locally is hard coded and should be made configurable. Will try and create a patch for this shortly.

marcoBauli’s picture

marcoBauli commented
Title: Allow for icons in GE to be altered » Allow for icons in GE to be altered (per c.type)
StatusFileSize
new custom_markers.diff3.98 KB

here's a patch that lets pick markers per content type

feedback appreciated ;)

raintonr’s picture

raintonr commented
Status: Needs review » Reviewed & tested by the community

Re: #6. Tested this on our system with latest KML (5.x-1.x-dev (2007-Nov-20))

Patch applied and worked perfectly. Nice work, would like to see this committed into main KML code.

raintonr’s picture

raintonr commented
Status: Reviewed & tested by the community » Fixed

Committed this patch to CVS just now.

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.