Bojanz pointed out some spacing issues, particularly on the product list and product display pages. I noticed that not all the classes are the same when a store is installed with the "no demo store" option, which uses this theme. It looks like the structure is slightly different causing the different class names.

CommentFileSizeAuthor
#1 1710682-class-changes.patch2.08 KBdudenhofer
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dudenhofer’s picture

dudenhofer CreditAttribution: dudenhofer commented
Status: Active » Needs review
FileSize
1710682-class-changes.patch2.08 KB

Attached a patch that should change the issues noted above.

bojanz’s picture

bojanz CreditAttribution: bojanz commented

-.node-product-type .field-name-field-product {
+.node-product-type .field-name-field-product-variations {

We should be naming the fields consistently, they should not be different between demo store and no demo store...

Cookiz’s picture

Cookiz CreditAttribution: Cookiz commented
Status: Needs review » Fixed

Fixed,
http://drupalcode.org/project/omega_kickstart.git/commit/7ae1371303b18f7...

@bojanz, I've bring these changes into commrece_kickstart theme too, https://code.drupalcommerce.org/#/c/281/

Status: Fixed » Closed (fixed)

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