jQuery Form has had some updates since February. 2.33 includes a number of bug fixes as well as huge commit for event delegation of the click handler.

This patch uses Dean Edwards' Packer on jQuery Form 2.33 with both base64 encoding and shrink variable names enabled.

CommentFileSizeAuthor
#8 update-jquery-form_609622_8.patch18.09 KBmfer
#6 609622_jqform.patch19.07 KBandypost
#5 609622.patch18.99 KBrobloach
#4 609622.patch18.33 KBrobloach
#1 jqueryform.patch15.84 KBcasey
jqueryform.patch15.71 KBrobloach

Comments

casey’s picture

casey commented
StatusFileSize
new jqueryform.patch15.84 KB

Update to 2.36

brianV’s picture

brianV commented
Status: Needs review » Reviewed & tested by the community

May as well get this in, although I figure we will update again right before release.

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Status: Reviewed & tested by the community » Needs review

Can we please use JSMin (used in most other jQuery JS) instead of Packer?

robloach’s picture

robloach
he/him
commented
StatusFileSize
new 609622.patch18.33 KB

Definitely, although it would be a little bit larger file, it would improve JavaScript load speed performance.

robloach’s picture

robloach
he/him
commented
StatusFileSize
new 609622.patch18.99 KB

Whoops, forgot CHANGELOG.txt.

andypost’s picture

andypost
he/him
Primary language Russian
commented
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new 609622_jqform.patch19.07 KB

+1 to commit, but could someone to test this change for #240777: Attach: An HTTP Error 0 occurred (on file upload)
Personally I have no issues under FF36, IE8, Opera 10.10 - tested with imagefield

Patch applied with only on hunk so I reroll

patching file modules/system/system.module
Hunk #1 succeeded at 1089 (offset 3 lines).

Status: Reviewed & tested by the community » Needs work

The last submitted patch, 609622_jqform.patch, failed testing.

mfer’s picture

mfer commented
Status: Needs work » Needs review
StatusFileSize
new update-jquery-form_609622_8.patch18.09 KB

The previous patch had some line breaks in the minified file that should not have been there. This patch removes those. The failed tests passed for me locally with the change.

andypost’s picture

andypost
he/him
Primary language Russian
commented
Status: Needs review » Reviewed & tested by the community

Suppose my patch was done with wrong params...

mcrittenden’s picture

mcrittenden commented

Sub.

dries’s picture

dries commented
Status: Reviewed & tested by the community » Fixed

We'll continue to sync with the latest jQuery releases for a while. Committed to CVS HEAD. Thanks.

Status: Fixed » Closed (fixed)

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