Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

Absolute minimum jQuery 1.9 patch #613

Closed
wants to merge 2 commits into from

2 participants

@seriema

This is the minimal needed to upgrade to jQuery 1.9.

@jzaefferer
Owner

Thanks! Its interesting that only the second attr call needs to be changed to make the test pass...

@seriema

The first use is wrong IMHO because the jQuery docs recommend either attr('checked', 'checked') or prop('checked', true). Regardless it should be symmetric use of either attr or prop.

@seriema seriema referenced this pull request from a commit
Commit has since been removed from the repository and is no longer available.
@seriema seriema referenced this pull request from a commit
Commit has since been removed from the repository and is no longer available.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.