Skip to content

Correct IE note on removeAttr page #642

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
Closed
markelog opened this issue Feb 2, 2015 · 2 comments
Closed

Correct IE note on removeAttr page #642

markelog opened this issue Feb 2, 2015 · 2 comments
Assignees
@arthurvr

Comments

@markelog
Copy link
Member

markelog commented Feb 2, 2015

Talking about this one -

Note: Removing an inline onclick event handler using .removeAttr() doesn't achieve the desired effect in Internet Explorer 6, 7, or 8. To avoid potential problems, use .prop() instead:

Aside of removing Internet "Explorer 6, 7" for 3.0, it should include 9 and 11, but not 10.
@arthurvr arthurvr self-assigned this Feb 9, 2015
@arthurvr arthurvr mentioned this issue Feb 11, 2015
Closed
@arthurvr
Copy link
Member

@markelog I opened #652, could you review?

arthurvr added a commit that referenced this issue Feb 12, 2015
@arthurvr
removeAttr: update note on support in IE
3cec29b 
Fixes gh-642
Closes gh-652
@arthurvr
Copy link
Member

Merged in 3cec29b. GitHub didn't close the PR/issue since it's merged into the v3 branch, not master.

arthurvr added a commit that referenced this issue Apr 9, 2015
@arthurvr
removeAttr: update note on support in IE
71d8a08 
Fixes gh-642
Closes gh-652
AurelioDeRosa pushed a commit that referenced this issue May 23, 2016
@arthurvr @AurelioDeRosa
removeAttr: update note on support in IE
69aefdc 
Fixes gh-642
Closes gh-652
timmywil pushed a commit that referenced this issue Jun 9, 2016
@arthurvr @timmywil
removeAttr: update note on support in IE
353fb32 
Fixes gh-642
Closes gh-652
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@arthurvr arthurvr

Labels
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@markelog @arthurvr