This issue is to address the suggestion in #1108348: User interface improvements that we make the granularity select list on the field settings form into a set of checkboxes. Patch forthcoming.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

ksenzee’s picture

Status: Active » Needs review
FileSize
1.95 KB
25.01 KB

This patch uses the same technique as the user edit page for creating a required disabled checkbox.

ksenzee’s picture

Oops. Just noticed this patch breaks the validation that requires fields without hours granularity to have timezone handling of 'none'. Here's a new patch, with a test for the validation. Also attaching an interdiff showing just this change, in case anyone reviewed the first patch already.

KarenS’s picture

Status: Needs review » Fixed

Looks good and I couldn't find a way to break it, so I have committed it.

Thanks!

Status: Fixed » Closed (fixed)

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