WordPress.org

Forums

P2
Patch for jQuery -> $ and some closure scoping (4 posts)

  1. vhauri
    Member
    Posted 3 years ago #

    I've got a patch that I ended up with after hacking P2 last week that cleans up some of the jQuery references by passing $ in the document.ready calls where appropriate and replacing the jQuery instantiation with $ inside those closures. It also moves a couple of functions around in the inc/options-page.php JS. Is there somewhere I can submit this patch (doesn't look like the default WP.org credentials allow opening of trac tickets in the themes repo)?

    Gist for the diff: https://gist.github.com/1525319

    Thanks,

    -Vasken

    http://wordpress.org/extend/themes/p2/

  2. Lance Willett
    Themer
    Posted 3 years ago #

    Thanks vhauri, this is the correct place to submit P2 theme suggestions.

  3. vhauri
    Member
    Posted 3 years ago #

    @Lance,

    Thanks for the info (and for inspiring me to update my profile info).

    -Vasken

  4. Michael Fields
    Themer
    Posted 3 years ago #

    Thanks for the patch. Much of this code has made it's way into the next version of P2 (soon-to-be-released).

Topic Closed

This topic has been closed to new replies.

About this Theme

About this Topic

Tags