-
Notifications
You must be signed in to change notification settings - Fork 8
Open
Description
Wanted to let you know of a problem we encountered. We had to make a change to your jquery plugin on our local site which was used in a WordPress theme called Jobify. It caused some errors on the site and through debugging we discovered the issue.
The problems we encountered were:
- It assumes that the ‘$’ variable is set to jQuery, which is no longer the case for modern versions of Wordpress.
- The plugin is coded to execute on “DOM Ready” rather than being wrapped in an IIFE, as is common for jQuery plugins.
Both issues are remedied by coding the plugin in accordance with http://learn.jquery.com/plugins/basic-plugin-creation/. Particularly, the section under “Protecting the $ Alias and Adding Scope”.
The website can be viewed at sfsummerjobs.org.
Metadata
Metadata
Assignees
Labels
No labels