Ref: http://blog.jquery.com/2012/08/09/jquery-1-8-released/

jQuery core 1.8.0 is compatible with the latest versions of jQuery UI (1.8.22), and jQuery Mobile (1.1.1).

Even though it doesn't say its NOT compatible but it is not compatible.

So if we enable jquery core 1.8 through jquery update the ui would be still at 1.8.11. Jquery ui 1.8.11 is not compatible with jquery 1.8 due to deprecated curCss function. So i suggest jqery_update also ship with an option to include jquery ui 1.8.22 atleast.

I would have marked this as feature request. But it creates some issues due to the above mentioned incompatibility ( for example it breaks ctools modal popups). So marking this is a bug.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

D34dMan’s picture

Version: 7.x-2.2 » 7.x-2.x-dev

sorry reported this issue in wrong branch... now its ok.

D34dMan’s picture

I also noticed that instead of giving different version upgrading ( i replaced all the files ) jquery ui to 1.8.22 solves the issue.

wiifm’s picture

Priority: Normal » Critical

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?

wiifm’s picture

Status: Active » Needs review
FileSize
690.35 KB

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

darrenmothersele’s picture

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

redben’s picture

Status: Needs review » Reviewed & tested by the community

Perfect. Thanks wiifm

rwinikates’s picture

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:

D34dMan’s picture

Status: Reviewed & tested by the community » Needs review

Changing the status since a new patch was uploaded in previous comment.

rickmanelius’s picture

Status: Needs review » Reviewed & tested by the community

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.

betz’s picture

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

timbrandin’s picture

#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.

timbrandin’s picture

Status: Reviewed & tested by the community » Needs work

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

marcvangend’s picture

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.

D34dMan’s picture

Status: Needs work » Needs review

So lets review it then :)

queenvictoria’s picture

Works for me.

rokk’s picture

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

marcvangend’s picture

Status: Needs review » Reviewed & tested by the community

@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.

damontgomery’s picture

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

My issue was specific to Media 2.x

Thanks!

hswong3i’s picture

Title: Give an option to select different jquery ui version to support jquery 1.8 » Update jQuery UI to 1.8.24
Status: Reviewed & tested by the community » Needs review
FileSize
1.58 MB

Please kindly correct me:

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

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 ;-)

hswong3i’s picture

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

marcvangend’s picture

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.

ericduran’s picture

Status: Needs review » Closed (duplicate)

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).

ericduran’s picture

Issue summary: View changes

Fixed a typo.