Attached is a patch. This does actually break compatibility with earlier jquery versions because it uses a method called .prop. I'm working on a backwards compatible patch.

Comments

MrMaksimize’s picture

StatusFileSize
new2.09 KB

Oops that one came up blank. here's a new and improved one.

pdrake’s picture

StatusFileSize
new1.88 KB

Here is a re-roll of this patch against the latest 6.x-2.x, relative to module root.

artfulrobot’s picture

StatusFileSize
new3.12 KB

Thanks! I'd found and fixed the missing quotes around the href selector, but missed the prop().

Here's my patch which includes the href fix, the prop fix, and adds a fix for the breadcrumbs mechanism (which used on a workaround for jquery 1.2 that didn't seem necessary with 1.7)

Against: 7.x-2.0-beta3

dcyuri7’s picture

thanks this helped me :)

jackbravo’s picture

Version:6.x-2.x-dev» 7.x-2.0-beta3
Status:Active» Reviewed & tested by the community

Yes this works for me with jquery_update and admin. Without this patch I get "Syntax error, unrecognized expression: [href=/admin]".

populist’s picture

StatusFileSize
new2.93 KB

Here is a quick reroll of #3 to cleanly apply. I also tested this and think its great.

idflood’s picture

Status:Reviewed & tested by the community» Needs review
StatusFileSize
new3.24 KB

Patch in #6 doesn't apply cleanly on 7.x-2.x so I manually rerolled it. There was two lines about the href which were already ok and I added a fix in admin.devel.js.

idflood’s picture

Version:7.x-2.0-beta3» 7.x-2.x-dev
idflood’s picture

StatusFileSize
new3.73 KB

New patch to make the modification compatible with jquery 1.4 and >= 1.6.