The "terms of service" element is supposed to set both 'form-type-webform-terms-of-service' and 'js-form-type-webform-terms-of-service' as CSS classes on the wrapper attributes, but overrides the non-js one because the "class" array key is set as string both times, instead using array syntax.

Patch is on the way...

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

agoradesign created an issue. See original summary.

agoradesign’s picture

Status: Active » Needs review
FileSize
768 bytes

here's the patch...

Status: Needs review » Needs work

The last submitted patch, 2: fix_terms_of_service_css_classes-2925530-2.patch, failed testing. View results

agoradesign’s picture

ok, seems the tests need to be corrected as well - they are already expecting the wrong classes

agoradesign’s picture

Status: Needs work » Needs review

jrockowitz’s picture

Status: Needs review » Fixed

Thanks!

agoradesign’s picture

You're welcome :)

And thanks again for the amazing reaction time :)

Status: Fixed » Closed (fixed)

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