woocommerce / storefront

Official theme for WooCommerce
https://wordpress.org/themes/storefront/
959 stars 472 forks source link

Remove max-width on image div for Mix and Match Products #2123

Closed helgatheviking closed 9 months ago

helgatheviking commented 10 months ago

Closes #2122.

Screenshots

How to test the changes in this Pull Request:

  1. Create a mix and match product.
  2. Set the customizer setting to grid display and number of columns to 3.
  3. The grid should look the same with or without the rule in question.

Changelog

Removed some unnecessary style in the Mix and Match CSS integration

danieldudzic commented 9 months ago

@helgatheviking thank you for this PR.

Please update the testing steps and we'll review.

helgatheviking commented 9 months ago

Create a mix and match product. Set the customizer setting to grid display and number of columns to 3. the grid should look the same with or without the rule in question, so I have removed it.

sunyatasattva commented 9 months ago

Hello @helgatheviking ! Thank you for your contribution and sorry to have kept you waiting. Please feel free to keep contributing and tell us more about the Mix and Match extension's needs. I've taken the liberty of editing the OP to have clearer testing steps and a changelog entry. Please, if you could, next time you contribute feel free to update these yourself, as it'd make the process much faster :)