Lots of responsive themes will add something like this (img max-width 100%) to their CSS to enable flexible images, while its their bad and easily fixed with:

.gmap img {
  max-width: none;
}

It could possibly be worth adding this to gmap.css, being a one line fix and probably saving a few bug reports to deal with. We ran into this in Adaptivetheme over here: #1297980: img max-width 100% breaks map displays - gmap, getlocations, openlayers

I'm adding the fix to our base theme, but I know of at least 3 or 4 other themes doing the max width trick including Zen 7.5 afaik.

Cheers.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

sunshinee’s picture

+1 for this request. I just spent a couple of hours searching Drupal.org to find such a simple solution.

eltioseba’s picture

Wow, thanks for that one.
I was having problems using Zen 7,5x (google map display as gray).

Zach Harkey’s picture

+1 for adding this to gmap.css

hkirsman’s picture

Title: img max-width 100% will break Google maps » img max-width 100% will break Google maps (in Firefox)
Version: 7.x-1.x-dev » 7.x-2.x-dev
Category: feature » bug
Status: Active » Needs review
FileSize
278 bytes

Here's the same thing in .patch file.

I added comment and important declaration.

Tested in FF 10 and FF 13.0.1

podarok’s picture

Status: Needs review » Needs work

#4

git apply --check gmap-img_max_width_100_breaks_firefox-1298800-1.patch
error: patch failed: gmap.css:61
error: gmap.css: patch does not apply

needs reroll against latest head

hkirsman’s picture

Status: Needs work » Needs review

You've commited it in another issue. I opened it up again because there's a typo: http://drupal.org/node/463160#comment-6692312

podarok’s picture

Status: Needs review » Fixed

#6 fixed in 6.x and 7.x branches

thanks!

Status: Fixed » Closed (fixed)

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

Anonymous’s picture

Issue summary: View changes

bad English, cleaning up