I was testing out this module today and ran into an issue on admin pages.
Since admin pages usually use an older version of jQuery the "on" function is not supported.
I added an if statement that checks for the function and cleaned up the spacing a little.

Comments

gooddesignusa’s picture

anybody’s picture

Thanks a lot! I'll have a look and patch that! :)

anybody’s picture

Status: Active » Closed (fixed)

Thanks a lot. I've investigated a bit and found a hopefully best solution which uses context and the .bind event handler instead of the .on function. I've released a new dev and stable version.

  • Commit 4759e5d on 7.x-1.x by Anybody:
    Now uses .bind and Drupals context variable instead of the jQuery .on()...