Closed (fixed)
Project:
Openlayers
Version:
6.x-2.x-dev
Component:
OpenLayers Views
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
8 Mar 2010 at 23:01 UTC
Updated:
20 Jun 2010 at 19:20 UTC
Jump to comment: Most recent file
Comments
Comment #1
zzolo commentedPlease note that this is only the case for grouped output.
Comment #2
zzolo commentedHere is a patch. I dont have time to fully test (as there is a fair amount of work in setting it up to test)
It's fairly straightforward. One thing I did change was the CSS class names to be more Drupal friendly and to use consistent naming. @tmcw, I have a feeling that Managing News actually uses these selectors, but its just a guess?
Comment #3
tmcw commentedI'd assume other sites use those classes as well. Just include both classes.
Comment #4
tmcw commentedThis patch is missing a semicolon in the theme function, so it breaks Drupal at the bootstrap level. Please fix, and then retest, and then I think that this will be commit'able.
Comment #5
zzolo commentedUpdated patch with semicolon and current branch. Don't have much time again to set it up, so maybe someone already has it set up.
Comment #6
zzolo commentedComitted:
http://drupal.org/cvs?commit=376098
Comment #7
tmcw commentedRe: #3, I've added the other class in. Otherwise this needs a changelog entry. Unannounced, unnoted compatibility breakages are not all right.