The subscription form submit button should be wrapped within the actions wrapper, as is customary on Drupal forms.

See https://api.drupal.org/api/drupal/developer!topics!forms_api_reference.h...

Patch forthcoming...

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

joelstein created an issue. See original summary.

joelstein’s picture

Status: Active » Needs review
FileSize
577 bytes
carsonw’s picture

Status: Needs review » Reviewed & tested by the community

+1

  • jami committed c019fe6 on 7.x-3.x authored by joelstein
    Issue #2554175 by joelstein: Put subscription form submit button inside...
jami’s picture

Version: 7.x-3.x-dev » 7.x-3.4
Status: Reviewed & tested by the community » Fixed

Looks good. Applied. Thank you!

joelstein’s picture

Thanks!

jami’s picture

Version: 7.x-3.4 » 7.x-3.x-dev

Status: Fixed » Closed (fixed)

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

Status: Closed (fixed) » Needs work

The last submitted patch, 2: mailchimp-submit-actions-wrapper-2554175-2.patch, failed testing.

Greg Boggs’s picture

Status: Needs work » Fixed
jami’s picture

Version: 7.x-3.x-dev » 7.x-3.4

Status: Fixed » Needs review

  • jami committed c019fe6 on 7.x-4.x authored by joelstein
    Issue #2554175 by joelstein: Put subscription form submit button inside...

  • jami committed c019fe6 on 8.x-1.x authored by joelstein
    Issue #2554175 by joelstein: Put subscription form submit button inside...
idebr’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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