Update jQuery UI to 1.8.24

Bumping priority as using jQuery 1.8 now should be bundled with jQuery UI 1.9

Error message when trying to use jQuery UI draggable:

Uncaught TypeError: Object function (a,b){return new p.fn.init(a,b,c)} has no method 'curCSS' 

There is a bug on jQuery about this, which is closed, as it is fixed in jQuery 1.9.0.

Anyone know any workarounds, or should one simply hack the new version in here?

Here is a patch to update jQuery UI to 1.8.24 which also resolves my issue with 'curCSS'

Thank you. The patch in #4 resolves the curCSS error I was getting from the Media module media browser popup.

Perfect. Thanks wiifm

Patch in #4 works if using local jquery, not if using CDN, so I subsumed that patch into mine which also increments the versions grabbed via CDN:

While I'm not going to do a line by line review of all the plugins... I did apply the patch and everything is working as expected. Also, the CDN edits looks right.

If you want more reviews before committing, no worries.

I can confirm that this patch fixes the 'curCSS' issue.

#1847900-7: Update jQuery UI to 1.8.24 has a couple trailing space errors when I applied it, but otherwise it helpes with ctools and media module.
These errors:

jquery_update-ui_update-1847900.patch:2225: trailing whitespace.
});
jquery_update-ui_update-1847900.patch:2305: trailing whitespace.
'ج',
jquery_update-ui_update-1847900.patch:2314: trailing whitespace.
'ج',
jquery_update-ui_update-1847900.patch:3260: trailing whitespace.
'ج',
jquery_update-ui_update-1847900.patch:3269: trailing whitespace.
'ج',
warning: squelched 24 whitespace errors
warning: 29 lines add whitespace errors.

Changing status, because the whitespace problems is causing a lot of output when building the site through drush.

Here is the same patch as in #7, but with all whitespace errors solved. (Yay! for regex search and manually editing patch files :-))

If it works for you like it does for me, I guess we can set this back to RTBC.

Works for me.

Also works for me. You should use 7.x-2.x-dev version.

@rokreativa: yes, that's why this issue says "Version: 7.x-2.x-dev".

Given the positive feedback and the fact that this was RTBC before I fixed the whitespace errors in the patch, I'm setting this to RTBC again.

#1847900-13: Update jQuery UI to 1.8.24 seems to work great for me.

My issue was specific to Media 2.x

Thanks!

Please kindly correct me:

• We should also update the replace/ui/version.txt text as 1.8.24
• When manually compare downloadable package from http://blog.jqueryui.com/2012/09/jquery-ui-1-8-24/ I found that #13 didn't correctly replace ALL files as that from jquery-ui-1.8.24

Patch recreate with following procedures, so should include both #13 progress for CDN + replace all possible jquery-ui files as version 1.8.24:

• git clone --branch 7.x-2.x http://git.drupal.org/project/jquery_update.git
• patch -p1 < ../jquery_update-ui_update-1847900-13.patch
• rsync -av --delete ../jquery-ui-1.8.24/ replace/ui
• echo "1.8.24" > replace/ui/version.txt

Assume replace all jquery-ui's file as 1.8.24 is correct, this patch should coming with identical effect as that of #13. If this works as expected we should safely switch back as RTBC ;-)

@marcvangend: how do you solve the whitespace errors at #13? I try numbers of way but still fail for #19 :-(

hswong3i, I opened the patch file in a text editor (UTF-8 supporting, running on Linux - not sure if that matters here) and looked up the line numbers that git complained about. It turned out that those lines were doing weird things with spaces, tabs and non-latin characters that I don't understand. I removed trailing spaces and adjusted the indents to match the lines before and after.

So there's no need to support 1.8.24. Since there is now a 1.10.2 version. I'm trying to close all the UI related issues and merge them into one issue.

The issue is #1947438: Update jQuery UI to the latest version (1.10.2).

Fixed a typo.