We needed to select date output in css regardless of it being a single, range or interval.

Single and interval have a wrapper but range doesn't. This patch adds a wrapper for range output and adds the wrapper class "date-display-wrapper" for single and interval wrapper.

CommentFileSizeAuthor
#6 add-consistent-wrapper-selectability-6.patch2.38 KBpontus_nilsson
add-consistent-wrapper-selectability.patch2.45 KBemattias
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

cimo75’s picture

cimo75 CreditAttribution: cimo75 commented

+1 should go in

karel010’s picture

karel010 CreditAttribution: karel010 commented

+1 Really needs to be in there.
Just try to do a float on the dates. You will find that you can't do anything with the "to" seperator of ranged dates.

podarok’s picture

podarok
🇺🇦 he/him/his
Primary language Ukrainian
Location Ukraine
CreditAttribution: podarok commented
Issue summary: View changes
Status: Needs review » Needs work
Issue tags: +Needs tests

all feature requests should be covered by tests

podarok’s picture

podarok
🇺🇦 he/him/his
Primary language Ukrainian
Location Ukraine
CreditAttribution: podarok commented
Status: Needs work » Needs review

add-consistent-wrapper-selectability.patch queued for re-testing.

Status: Needs review » Needs work

The last submitted patch, add-consistent-wrapper-selectability.patch, failed testing.

pontus_nilsson’s picture

pontus_nilsson
Location Stockholm
CreditAttribution: pontus_nilsson commented
FileSize
add-consistent-wrapper-selectability-6.patch2.38 KB

Re-roll against latest in 7.x-2.x

Valentine94’s picture

Valentine94
Location Kyiv, Ukraine
CreditAttribution: Valentine94 commented
Status: Needs work » Needs review

Test

Status: Needs review » Needs work

The last submitted patch, 6: add-consistent-wrapper-selectability-6.patch, failed testing.