color is set to be disabled twice. no error but not needed.

CommentFileSizeAuthor
#1 2003594-_form_test_disabled_elements.patch590 bytesgcassie
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

gcassie’s picture

gcassie CreditAttribution: gcassie commented
Status: Active » Needs review
FileSize
2003594-_form_test_disabled_elements.patch590 bytes

and patch.

Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous commented
Status: Needs review » Reviewed & tested by the community

One liner. Very clean.

Status: Reviewed & tested by the community » Needs work

The last submitted patch, 2003594-_form_test_disabled_elements.patch, failed testing.

Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous commented
Status: Needs work » Needs review

#1: 2003594-_form_test_disabled_elements.patch queued for re-testing.

Status: Needs review » Needs work

The last submitted patch, 2003594-_form_test_disabled_elements.patch, failed testing.

Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous commented
Status: Needs work » Needs review

#1: 2003594-_form_test_disabled_elements.patch queued for re-testing.

Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous commented
Status: Needs review » Reviewed & tested by the community

Thanks Testbot. Glad you decided to see things my way.

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott commented
Status: Reviewed & tested by the community » Fixed

One less line of code to maintain :)

Committed d46f4d2 and pushed to 8.x. Thanks!

Status: Fixed » Closed (fixed)

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