The progress bar should have a class identifying the number of pages so that it can be styled differently and the widths can be adjusted dynamically

CommentFileSizeAuthor
#3 commerce_cart_progress_class.patch1.19 KBscotthooker
#1 commerce_cart_progress_class.patch1.19 KBscotthooker
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

scotthooker’s picture

scotthooker CreditAttribution: scotthooker commented
FileSize
commerce_cart_progress_class.patch1.19 KB

Patch attached to add a class depending on the number of pages.

scotthooker’s picture

scotthooker CreditAttribution: scotthooker commented
Status: Active » Needs review
scotthooker’s picture

scotthooker CreditAttribution: scotthooker commented
FileSize
commerce_cart_progress_class.patch1.19 KB

Patch attached to add a class depending on the number of pages.

applicity_sam’s picture

applicity_sam CreditAttribution: applicity_sam commented

Looks like a good solution to the outstanding issue of sizing the progress bar to the theme when it has a variable number of steps.

Tested the patch in #3 and it works for me.

chrisjlee’s picture

chrisjlee CreditAttribution: chrisjlee commented

Patch works for me. Thanks!

dwkitchen’s picture

dwkitchen CreditAttribution: dwkitchen commented
Status: Needs review » Reviewed & tested by the community
jcisio’s picture

jcisio CreditAttribution: jcisio commented
Issue summary: View changes
Status: Reviewed & tested by the community » Fixed

Thanks. Committed with come coding fixes: b1493e0.

Status: Fixed » Closed (fixed)

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