The 2026 Drupal Business Survey is open. Your response will help shape the future of the Drupal business ecosystem.Simple GMAP has had a few releases since the version in Panopoly:
- https://www.drupal.org/project/simple_gmap/releases/7.x-1.3
- https://www.drupal.org/project/simple_gmap/releases/7.x-1.4
I've also posted a patch to improve accessibility by adding a configurable iframe title to the formatter output: #2902178: Add title attribute to iframe (7.x). Once that's finished being reviewed, it'd be nice to get all this in to Panopoly.
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | panopoly_widgets-update_simple_gmap-2903251-3.patch | 1.36 KB | byronveale |
Comments
Comment #2
dsnopekHere's a patch for this!
EDIT: Here's a Travis build: https://travis-ci.org/panopoly/panopoly/builds/323068326
Comment #3
byronveale commentedPanopoly Widgets feature was coming up overridden; this took care of it for me locally…
Comment #4
bkosborneWe've had this in production for weeks w/o issue.
Comment #5
dsnopekHere's a Travis build for the latest patch:
https://travis-ci.org/panopoly/panopoly/builds/353055135
Mainly need to verify that that'll fix the overridden issue on the last build
Comment #6
dsnopekI did some testing of the Map widget with this update and everything worked great! Mainly, I'm just waiting for a successful test run, and then this'll get merged.
Comment #7
dsnopekTroubles with the tests! Here's a new build, where at least the update build should work:
https://travis-ci.org/panopoly/panopoly/builds/353489056
Comment #9
dsnopekThe test failures here are the same as happening on all tests:
There's nothing map specific in there. So, I'm just going to go ahead and commit this one!