Fix appearance of marker for required date fields

The required marker is problematic because we can have such a multitude of date sub-parts and the system error processing can interfere with what the date module is trying to do. I've tried handling this in different ways over the years, with mixed results. Plus not all sub-parts are required, even within dates that are required.

Sub.

Each use case needs to be defined and then the correct 'required' options be made available & labeled accordingly.

One simple use case that could be solved quickly: when the time field is not displayed and the date field is required, the main element's label should have the 'required' indicator. It may also be useful to remove the extra fieldset that gets added for this use case, but that's a separate issue.

If this helps anyone, I needed the required asterisk to appear on the fieldset legend, so hacked it via a bit of code in a custom module:

/**
 * Implements hook_form_alter()
 */
function CUSTOMMODULENAME_form_form_alter(&$form, &$form_state, $form_id) {
  if($form_id == 'ID_OF_FORM_HERE') {
    $form['ID_OF_FIELD_HERE']['und'][0]['#title'] .= '<span title="This field is required." class="form-required">*</span>';
  }
}

Attached is a patch that will add the red marker (form-required span) to the fieldset title in theme_date_combo.

The patch at #7 works perfectly for me.

The patch fixes the appearance of the marker indeed! Thank you. Do we need more review?

I'm still experiencing a possibly related issue: http://drupal.org/node/301538: Date field does not use #required to mark required date fields.

Does not work for me: the patch does add a required star to the fieldset legend (as expected) but does not remove the other star for me.

This can be fixed using CSS but ideally the other star should be removed...

patch in #7 works perfectly for me too.

guillaumev, can you provide steps to reproduce your issue on a clean install?

Attached is a patch that removes the floating marker and adds it to the label.

#12 works great for me against 7.x-2.6+0-dev

Can I ask why is the 'from' label/title is hidden/empty?

Kenneth's patch makes it not just empty but hides it completely.

In case of having starting and ending dates, I think label is quite essential for both.

Thanks,
David

It works for me over 7.x-2.6. Thanks.

Applied patch in #12. Works as desired to make a required day-month-year date with select lists display the required marker.

Placing the marker in the fieldset is considerable usability problem.

Since there are limitations for a systematic fix until now (#3), is there any possible hardcoded solution similar to #6 but applied to a the field label (rather than its fieldset) of a single date field ?

Thanks!

#12 works, thanks!

+1 to #12, thanks!

Applied #12 patch, it did the trick for us and so far no side effects.

#12: add-required-marker-1248786-12.patch queued for re-testing.

Committed to 7.x-2.x.

+1 to #12, great! thanks!