Skip to content
This repository has been archived by the owner on Oct 2, 2019. It is now read-only.

make sure sortable classes are removed properly #902

Closed
wants to merge 1 commit into from

Conversation

homerjam
Copy link
Contributor

@homerjam homerjam commented May 2, 2015

No description provided.

@wesleycho
Copy link
Contributor

Needs rebasing, but otherwise LGTM

@homerjam
Copy link
Contributor Author

@wesleycho I have rebased now. Thanks for ploughing through all the PRs, great work!

@user378230
Copy link
Contributor

You've just slightly borked the commit message now 😝

Can you fix? Is there a related issue for this?

@homerjam
Copy link
Contributor Author

Oops ☺️

No there's no issue - I just came across it and submitted the PR.

@wesleycho
Copy link
Contributor

@user378230 what are your thoughts - I can merge this locally and alter the message before pushing up if it makes it easier.

@user378230
Copy link
Contributor

@wesleycho LGTM now 👍 feel free to do it locally

@wesleycho wesleycho closed this in 4b1ed47 Apr 6, 2016
fcaballero pushed a commit to fcaballero/ui-select that referenced this pull request Apr 17, 2016
- Correctly remove sortable classes from elements on `dragend`

Closes angular-ui#902
fcaballero pushed a commit to fcaballero/ui-select that referenced this pull request Apr 25, 2016
- Correctly remove sortable classes from elements on `dragend`

Closes angular-ui#902
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants