Sometimes map needs to be altered by a module or a theme but it isn't possible at the moment because Leaflet module renders map itself. For example it's quite tricky to remove from page some leaflet js files added with drupal_add_js(). I propose use Render API and return to theme layer a renderable array not markup. It's a common practice in Drupal core modules.

CommentFileSizeAuthor
#1 build_map-1814028-0.patch4 KBchi

Comments

chi’s picture

chi commented
Status: Active » Needs review
StatusFileSize
new build_map-1814028-0.patch4 KB
gmclelland’s picture

gmclelland commented
Status: Needs review » Needs work

@Chi - I was going to test your patch, but it no longer applies.

gcb’s picture

gcb
he/him
Location Portland, OR
commented
Issue summary: View changes
Status: Needs work » Fixed

Nice idea. Worked this into the dev version.

DerTobi75’s picture

DerTobi75 commented

Hi,

sorry, for bringing this Issue up again.

Just one question, is it possible that this patch stops the Leaflet Markercluster Module working?

When using the Leaflet Markercluster Module with the latest Dev Version of Leaflet, no clustering librariefiles are loaded, ...

Regards,

Tobi

gcb’s picture

gcb
he/him
Location Portland, OR
commented
Status: Fixed » Closed (fixed)

@DerTobi75 if you are seeing another issue go ahead and create a separate ticket for it!

This fix is now in a release https://drupal.org/node/2148175