Hi, this wasn't re-enabling the .checkout-continue button for me with jQuery 1.11, so I changed the .attr() call to .prop() and it works. I also had to pass $clone to the enableContinue callback, not $btn, since $btn is the the hidden button. Patch attached.

CommentFileSizeAuthor
#3 commerce_clientside_validation-fix_enable_button-2548573-3.patch1017 byteseric.chenchao
#2 commerce_clientside_validation-fix_enable_button-2548573-1.patch1.22 KBbgilhome

Comments

bgilhome created an issue. See original summary.

bgilhome’s picture

bgilhome commented
StatusFileSize
new commerce_clientside_validation-fix_enable_button-2548573-1.patch1.22 KB
eric.chenchao’s picture

eric.chenchao commented
StatusFileSize
new commerce_clientside_validation-fix_enable_button-2548573-3.patch1017 bytes

@bgilhome Thanks for the patch. Just one thing I am not quite sure, as jQuery.prop is not available in jQuery 1.4.4 so I think jQuery.attr is more compatible?

  • 74894a0 committed on 7.x-1.x 
    Issue #2548573 by bgilhome, eric.chenchao: Use .prop instead of .attr,...
eric.chenchao’s picture

eric.chenchao commented
Status: Needs review » Fixed

@bgilhome thanks for the help I have commited the fix.

Status: Fixed » Closed (fixed)

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