FlorianMielke / FMMoveTableView

Discontinued: Please use iOS 11's Drag & Drop capability instead.
MIT License
623 stars 101 forks source link

- (void)prepareForMove method now works for custom UITableViewCells #12

Closed gtmtg closed 11 years ago

gtmtg commented 11 years ago

In the past, custom cells that don't use the properties textLabel, detailTextLabel, etc. or those that had a custom background color would remain visible when picked up (to move). I changed the FMMoveTableViewCell's - (void)prepareForMove method to work even for such cells.

FlorianMielke commented 11 years ago

Thank you for your pull request. The prepareForMove method's intent is to be overwritten by subclasses - which every cell of FMMoveTableView should be. So you can add everything in there that causes your custom table view cell to be clean for the move. Therefore I wont change the current behavior.

treddy73 commented 11 years ago

I'm curious...is there a reason why FMMoveTableViewCell can't be a category on UITableCellView and then you just check to see if the cell responds to the prepareForMove selector? It seems I shouldn't need to change all of my table cells to inherit from this class when a category will work just fine?