Closed (fixed)
Project:
Openlayers
Version:
6.x-2.x-dev
Component:
OpenLayers UI
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
7 Jun 2010 at 22:40 UTC
Updated:
3 Jan 2014 at 01:42 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
tmcw commentedLooks good. I'll see if ctools dependencies will work for this.
Comment #2
zzolo commentedJust to let you know. The Ctools dependency stuff is in there, at least from what I understood of the documentation. But fieldsets behave a little differently, apparently.
Comment #3
zzolo commentedFigured it out. You can't actually use a fieldset type, it has to be a hidden type with a prefix (and then another markup type for the suffix). Quite a hack, but still simpler that writing from scratch. I think this is a pretty good improvement.
Comment #4
zzolo commentedtagged as beta blocker
Comment #5
tmcw commentedHow is this a beta blocker? It's a good but non-essential improvement.
Comment #6
zzolo commentedBecause its in the queue ready to be reviewed and I would like to see it in. I just don't see the point in putting this off as it is really close to finishing.
Comment #7
tmcw commentedOkay, tested this and it work as advertised, feel free to commit.
Comment #8
zzolo commentedThanks for the review. Committed:
http://drupal.org/cvs?commit=416560