in marker.js is:
if (marker.text) {

if think this is bug and should be instead:
if (marker.text.length > 30) {

because marker.text always exist and has:
<div class="gmap-popup"></div>

Comments

j0rd’s picture

j0rd commented

is always empty because it's reading the wrong variable.

Code from the gmap.module:

  // Add a class around map bubble contents.
  // @@@ Bdragon sez: Becw, this doesn't belong here. Theming needs to get fixed instead..
  if (isset($map['markers'])) {
    foreach ($map['markers'] as $i => $marker) {
      // Added in DPM from devel module
      dpm(array("MARKER $i" => $marker));
      $map['markers'][$i]['text'] = '<div class="gmap-popup">' . $marker['text'] . '</div>;
    }
  }

You'll notice from the dpm() that $marker['text'] is blank and those gmap-popup is always blank.

@bdragon: from reading your comments in the code, I can tell you know this stuff is horribly broken. Could we at least get a simple patch to fix up the theming layer of the gmaps module to reflect the documentation in THEMEING.txt, it wouldn't be much work and would help the 5000+ people who are using RC1.

I am very surprised to not find a lot of issues on this bug though. I assume most people are using the views integration which works fairly well.

riversidekid’s picture

riversidekid commented

Subscribe. I have updated CCK-6x-3.x-dev, ctools-6x-1.6, gmap-6x-1.1-rc1, location-6x-3.1-rc1, views-6x-2.11 and views-6x-3.0-alpha3. Nothing is helping. I tried the manual edits within http://drupal.org/node/457222#comment-3161880 to no avail as well. At this point, I hope it is a bug in the code as I don't have much hair left! ;)

johnv’s picture

johnv
Primary language Dutch
commented
Status: Active » Closed (won't fix)

Closing this very old issue. Please reopen if it is still valid.