Hi community,

drupal jquery version : 1.8
chrome : 24.0.1312.57

After installing admin module, no javascript work anymore ! I get on every page the following js error :

Uncaught Error: Syntax error, unrecognized expression: ul.menu a[href=/admin/content]

After looking at the jquery documentation (http://api.jquery.com/category/selectors/attribute-selectors/) we should surround selectors which contain not valid (http://www.w3.org/TR/CSS21/syndata.html#value-def-identifier) chars

Attribute values in selector expressions must follow the rules for W3C CSS selectors; in general, that means anything other than a valid identifier should be surrounded by quotation marks.

Lots of talk around escaping selectors on forums, it seems that for our needs here, simply quoting the string make jquery able to perform the job

Comments

Status:Active» Reviewed & tested by the community

Works for me - a site with admin and jquery update do not function as expected when js aggregation is turned on.

Status:Reviewed & tested by the community» Needs review
StatusFileSize
new1.63 KB

I incorporated 7gipsy's changes above using the more general git instructions for modules. Not everyone puts this module as a sub-folder of contrib, so doing that adds another step for testing it, and integrating it. Marking as needs review since this is technically a new patch, though its really the same thing.

The last patch works for me if I use jQuery 1.8, but not 1.7. I get:
"Uncaught TypeError: Cannot call method 'html' of undefined"
Line 90 of jquery.drilldown.js

StatusFileSize
new2.25 KB

This one seems to work with jQuery 1.5, 1.7 and 1.8. With jQuery 1.6+, attr() may return 'undefined' and break chaining, hence the error you got. See http://api.jquery.com/attr/

Patch seems to work fine. Thanks

Assigned:Unassigned» skwashd
Status:Needs review» Reviewed & tested by the community

I have tested this and it works as expected. I will play with it for a day or 2 to ensure it doesn't have any unintended side effects, but I hope to commit this by the end of the week.

Issue summary:View changes
StatusFileSize
new2.34 KB

Based on the patch from "mdupont" in #4 ive created a new patch which fixes the bug "settings.activeLink is undefined". It appears when you try to navigate thru the breadcrumb and breaks the navigation.

#7 seems to work, tested on jquery 1.5, 1.7 and 1.8