Skip to content

this commit fixes bug 7106; additionally it adds a test case for both issues in the bug.#503

Closed
ddstreet wants to merge 1 commit intojquery:masterfrom
ddstreet:master
Closed

this commit fixes bug 7106; additionally it adds a test case for both issues in the bug.#503
ddstreet wants to merge 1 commit intojquery:masterfrom
ddstreet:master

Conversation

@ddstreet
Copy link
Contributor

animateClass: fix to not overwrite css nor class settings changed during animation. Fixed #7106 - animateClass: css and class changes during animation are lost

…ing animation. Fixed #7106 - animateClass: css and class changes during animation are lost
@gnarf
Copy link
Member

gnarf commented Oct 22, 2011

Looks pretty good! Thanks for the time, this was basically the the exact method I was planning on using too.

The commit messages should be "Effects: ...." but thats minor enough I can handle it when I pull it in, just wanted you to know for future contributions. http://wiki.jqueryui.com/w/page/25941597/Commit-Message-Style-Guide

Thanks.

@ddstreet
Copy link
Contributor Author

The commit messages should be "Effects: ...."

Oops! I'll remember that for next time. Thanks!

@gnarf
Copy link
Member

gnarf commented Oct 25, 2011

Thanks, landed this here: e3156ea

@gnarf gnarf closed this Oct 25, 2011
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants