Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

make toggle(true/fales) and toggleClass("...", true/false) shortcut to show/hide and add/removeClass #1271

Closed
wants to merge 3 commits into from
Closed

make toggle(true/fales) and toggleClass("...", true/false) shortcut to show/hide and add/removeClass #1271

wants to merge 3 commits into from

Commits on May 16, 2013

  1. avoiding jQuery(this) and a closure when calling .toggle(true/false)

    @jbedard
    jbedard committed May 16, 2013
    Copy the full SHA
    12f865b View commit details
    Browse the repository at this point in the history

  2. removing whitespace

    @jbedard
    jbedard committed May 16, 2013
    Copy the full SHA
    013727f View commit details
    Browse the repository at this point in the history

  3. directly calling add/removeClass when calling toggleClass(..., true/f… 

    …alse) to avoid jQuery(this) for each element
    @jbedard
    jbedard committed May 16, 2013
    Copy the full SHA
    affa2b4 View commit details
    Browse the repository at this point in the history