[Needs tests] Add more theming options to popup bubble by using infobubble library plugin

very creative module indeed !

+++ b/gmap_style_bubbles/gmap_style_bubbles.moduleundefined
@@ -0,0 +1,234 @@
+ * Anything before the first colon in each line is considered an array key, and
+ * anything following the first semicolon is that key's value.

"semicolon" should be "colon," right? Overall though, I'm pretty confused about what this function accepts. What exactly is it able to clean up / what's the ideal input? Is this being transformed into JSON or into actual JavaScript?

+++ b/gmap_style_bubbles/gmap_style_bubbles.moduleundefined
@@ -0,0 +1,234 @@
+  $options = array(
+    "shadowStyle" => t("Options: 0 (no shadow), 1 (default shadow), 2 (sharp shadow)"),
+    "padding" => t("Like the css property, the padding of the bubble."),
+    "borderRadius" => t("Like the css property, the radius of the bubble's corners."),
+    "borderWidth" => t("Like the css property, the width in pixels of the border."),
+    "borderColor" => t("Like the css property, the hex or rgb color of the border."),
+    "backgroundColor" => t("This is the background of the content of the bubble, but NOT the entire bubble itself."),
+    "bubbleBackgroundClassName" => t("A css class for the entire bubble."),
+    "minWidth" => t("Like the css property, the minimum width of the bubble."),
+    "maxWidth" => t("Like the css property, the maximum width of the bubble."),
+    "minHeight" => t("Like the css property, the maximum height of the bubble."),
+    "arrowSize" => t("The width of the pointer arrow. NOTE: if you choose an arrow style that only uses half of the arrow, this number will be twice as wide as your arrow at its widest point."),
+    "arrowPosition" => t("The percent from the left of the bubble where the arrow will appear."),
+    "arrowStyle" => t("Options: 0 (full triangle), 1 (half triangle leaning left), 2 (half triangle leaning right)"),
+    "closeImage" => t("The url of the image to use as the close button for the bubble"),
+  );

For coding standards, these should all be single quotes except where a t() string has a single quote inside of it.

+++ b/gmap_style_bubbles/js/infobubble_extended.jsundefined
@@ -0,0 +1,22 @@
+  InfoBubble.prototype['setBubbleBackgroundClassName'] =

What does this do? I think foo['bar'] and foo.bar are equivalent.

+++ b/gmap_style_bubbles/gmap_style_bubbles.infoundefined
@@ -0,0 +1,18 @@
+
+version = 7.x-1.0

This should be removed. It will be added by the Drupal.org packager - you don't have to specify versions manually in contrib modules.

+++ b/gmap_style_bubbles/gmap_style_bubbles.infoundefined
@@ -0,0 +1,18 @@
+
+
+; Information added by drush on 2013-07-05
+version = "7.x-2.7-rc1+3-dev"
+core = "7.x"
+project = "gmap"
+datestamp = "1373057363"

This should be removed.

Thank you very much for this patch, it is much needed to make customization of gmap complete.
I was using #774078: ExtInfoWindow for GMap - Styling the info window for Google Maps before but it hasn't been updated for Google Maps v3 and it seems that InfoBubble is its successor.

I'd like to suggest a few improvements, is it possible to add some properties for the close image to be able to position it and set its size as these setting are currently hardcoded in infobubble.js ? At least height, width and margin properties would help a lot.

  close.style['position'] = 'absolute';
  close.style['width'] = this.px(12);
  close.style['height'] = this.px(12);
  close.style['border'] = 0;
  close.style['zIndex'] = this.baseZIndex_ + 1;
  close.style['cursor'] = 'pointer';

I noticed that your modifying marker.js which would prevent this module to be a standalone module, I haven't seen some hook available there in gmap module but this would definitely be a great addition to the gmap project to avoir the need to hack.

Again thank you so much for this great work and write up.

infobubble_extended.js is a very nice clean way to improve infobubble, I'll be adding it to the Getlocations module for sure.

it doesnt work for me !!! i can see the fields and fieldset and i added this " backgroundColor: #222222" and nothing happens ! could you provide me some help. I have also included the library source files.

The interdiff should be named ".txt" to avoid being tested (creating an interdiff).
Thanks for this great patch.

#12 ok, let`s make it in
commited pushed to 7.x-2.x
this one needs test coverage before stable release, so making it critical task ans assigned to author
Thanks!!!

One minor typo and "maxHeight" added to function _gmap_style_bubbles_help_text()
Patch on dev attached

Still trying to fix my images

This causes a javascript error in IE 8 that breaks the map. Adding this javascript file in the theme fixes the issue: https://github.com/eligrey/classList.js/blob/master/classList.js

Example IE8 inclusion in an AdaptiveTheme theme's info file:

  ie_scripts[lt IE 9][] = scripts/classList.js

20: gmap-infobubble-style-2035847-20.patch queued for re-testing.

re #23
Your patch has broken tests
Look at screenshot from simpletest.me

It is impossible
You can only write unit tests for you code in such situations

20: gmap-infobubble-style-2035847-20.patch queued for re-testing.

Still it is like patch? or released any module for this?