It's just a ":" and nothing else. No span, no div, no classes. This makes it really difficult to style and fix issues with it.

I believe this should be easier than:

function mytheme_preprocess_webform_time(&$variables) {
  $variables['element']['minute']['#prefix'] = '<span class="webform-time-separator">:</span>';
}

Comments

hass’s picture

hass CreditAttribution: hass commented
Version: 7.x-3.9 » 7.x-3.19
quicksketch’s picture

quicksketch
he/him
CreditAttribution: quicksketch commented

Thanks, good suggestion. Adding a span like your example looks perfect.

DanChadwick’s picture

DanChadwick CreditAttribution: DanChadwick commented
Issue summary: View changes
Status: Active » Closed (won't fix)

Closing for lack of activity and demand. Also, 7.x-3.x is receiving critical bug fixes only.

hass’s picture

hass CreditAttribution: hass commented
Status: Closed (won't fix) » Active

I still need this and code has not committed as I know.

DanChadwick’s picture

DanChadwick CreditAttribution: DanChadwick commented
Status: Active » Closed (won't fix)

Reopen with a patch for 7.x-4.x if you still would like this change.

hass’s picture

hass CreditAttribution: hass commented
Version: 7.x-3.19 » 7.x-4.x-dev
Status: Closed (won't fix) » Active
DanChadwick’s picture

DanChadwick CreditAttribution: DanChadwick commented
Status: Active » Closed (won't fix)

Upon further reflection, webform doesn't need the markup bloat when there is so little demand for this and there is a theme workaround.