Problem/Motivation

When popup features are clustered, there is no distinctive class applied to each result, making basic styling difficult.

Proposed resolution

Provide simple classes that are typically included in standard Drupal practice when multiples of an item are output, such as rows.

Remaining tasks

Patch to be supplied, reviewed and tested.

CommentFileSizeAuthor
#2 openlayers-popup-feature-cluster-first-last-class-1559852-2.patch952 bytesanniegreens
#1 openlayers-popup-feature-cluster-first-last-class-1559852-1.patch854 bytesanniegreens
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

anniegreens’s picture

anniegreens CreditAttribution: anniegreens commented
Status: Active » Needs review
FileSize
openlayers-popup-feature-cluster-first-last-class-1559852-1.patch854 bytes

Patch attached that provides simple 'first' and 'last' class on popup features in a cluster.

anniegreens’s picture

anniegreens CreditAttribution: anniegreens commented
FileSize
openlayers-popup-feature-cluster-first-last-class-1559852-2.patch952 bytes

New patch accommodates one feature in a cluster, giving it both 'first' and 'last' classes.

Pol’s picture

Pol
the/that
Location Brussels
CreditAttribution: Pol commented
Status: Needs review » Fixed

I changed the code a bit, it's now committed.

http://drupalcode.org/project/openlayers.git/commit/cf6ab2a

Thanks.

Status: Fixed » Closed (fixed)

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