It'd be nice if the add to cart form used an actions wrapper around the submit button for easier theming. Patch attached.

CommentFileSizeAuthor
commerce_add_to_cart_actions-1.patch1.62 KBrbayliss
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

rszrama’s picture

Title: Use actions wrapper on add to cart form » Use an actions wrapper on the Add to Cart form
Status: Needs review » Postponed

Curious, I'm not sure I knew about this element type when we first started building our forms, as it appears we use containers for various actions elements. Unfortunately, such a change would be too disruptive to the 1.x branch - who knows how many modules or themes depend on the current array structure? Until we open the 2.x branch, this will just have to be altered in via a custom module.

rszrama’s picture

Version: 7.x-1.x-dev » 8.x-2.x-dev
Issue summary: View changes

Just need to make sure we get this in properly.

mglaman’s picture

Status: Postponed » Closed (outdated)

We are adding the add to cart button via the ::actions method and should be rendered appropriately (covering initial issue summary bases.)

Marking Closed (outdated) as "it just works" for 2.x and was deemed a breaking change for 1.x.