Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Hello,
with latest jQuery update dev release, with jQuery UI - v1.10.2 autocomplete deluxe fields don't render. Chrome inspector reports this:
Uncaught TypeError: Object [object Object] has no method 'menu'
Cheers
Comment | File | Size | Author |
---|---|---|---|
#7 | autocomplete_deluxe-2000308-7.patch | 554 bytes | pjcdawkins |
#5 | autocomplete-deluxe-renderitem-not-exist.patch | 640 bytes | MauPalantir |
#4 | resizemenu-doesnt-exist-2000308-4.patch | 429 bytes | queenvictoria |
#2 | resizemenu-doesnt-exist-2000308-2.patch | 475 bytes | queenvictoria |
Comments
Comment #1
Lux Delux CreditAttribution: Lux Delux commentedTurns out jquery Update, when using the new library, was not adding UI Menu component.
Manually including it on my admin pages solved the problem.
I'll report there, you can close this.
Cheers
Comment #2
queenvictoria CreditAttribution: queenvictoria commentedI've patched jQuery update over here to fix that. You'll need to flush caches afterward.
http://drupal.org/node/2002568#comment-7447018
But I was left with a different problem.
jqObject.data("autocomplete") doesn't exist and therefore I can't override the _resizeMenu method (which does still exist in ui.autocomplete--I've checked). So here is a patch for that. Needs work I suspect.
Comment #3
queenvictoria CreditAttribution: queenvictoria commentedComment #4
queenvictoria CreditAttribution: queenvictoria commentedI've had another go at this and used the same code style as the renderItem monkey patch. Please review.
Comment #5
MauPalantir CreditAttribution: MauPalantir commentedI applied the patch but autocomplete_deluxe still had a similar problem on line 122, with function _renderItem(). So I had to patch it like above. Now it works with latest dev jQuery Update. Attached the patch.
Comment #6
sepgil CreditAttribution: sepgil commentedYour patch doesn't apply since it seems to be a diff from core git and not acdx...
Comment #7
pjcdawkins CreditAttribution: pjcdawkins commentedTry this
Comment #8
M@ster CreditAttribution: M@ster commentedqueenvictoria thank you for patch. I'ts work
Comment #9
brockfanning CreditAttribution: brockfanning commented#7 is working for me, on a site running jQuery 1.8.
Comment #10
pjcdawkins CreditAttribution: pjcdawkins commentedComment #11
sepgil CreditAttribution: sepgil commentedThe patch works, but JQuery 1.8 produces another bug: #1881120: wrong position of ui-element in jquery version 1.8
So it would be nice if could fix that before committing this patch.
Comment #12
ckng#7 works for me
Comment #13
fullerja CreditAttribution: fullerja commentedPatch in #7 worked nicely for me. Can this get committed rather than wait for #1881120: wrong position of ui-element in jquery version 1.8?
Comment #14
claudiu.cristeaThis works, why not committing?
Comment #15
Nicolas Bouteille CreditAttribution: Nicolas Bouteille commented+1 RTBC
Comment #16
rcodina CreditAttribution: rcodina commentedPlease, also checkout the patch here: https://www.drupal.org/node/2276111
Comment #17
sepgil CreditAttribution: sepgil commentedI committed the patch from #2276111: Uncaught TypeError: Cannot read property 'msie' of undefined (jquery 1.10), so this issue should be fixed to. Thanks for all the contributions.