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.

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

Enabled drag & drop to an empty section #12

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

eltonf
Copy link

@eltonf eltonf commented Aug 26, 2013

My previous comment was partially incorrect. It appears that a section footer or header is not required for this to work.

eltonf and others added 3 commits August 25, 2013 08:39
…header)

In the original implementation it's impossible to drag & drop a cell into an empty section since the logic looks for an adjacent cell to derive an NSIndexPath. Because an empty section does not have any cells the logic looks for the CGRect of the header or footer while dragging the table cell only if it doesn't have a current match for a table cell. When there is a hit for a header or footer the next NSIndexPath is assumed to be section #, row 0 in the section that lacks any table cells.
@SwiftArchitect
Copy link

This is a great improvement!

Can we take action and make a new PodSpec? There seem to be 6 pending pull requests, yet no changes in 2 years...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants