search

bvogelzang / BVReorderTableView

Easy Long Press Reordering for UITableView
MIT License
234 stars 59 forks source link

Handle table views where contentInset.top != 0 which is often the case in iOS7 #7

Closed mluisbrown closed 11 years ago

mluisbrown commented 11 years ago

I've been trying this great little class with iOS7 (beta3) for a new project I'm working on. In iOS7, UITableViews extend up under the translucent navigation bar and status bar and therefore have a contentInset set where top is not 0 (it's usually 64, which is the height of the nav bar, 44 + the height of the status bar, 20). This class didn't handle the case where the UITableView had a contentInset.top other than 0 (regardless of whether running in iOS7 or not). These changes made the class handle a top content inset properly.

bvogelzang commented 11 years ago

Good fix. Thanks for the contribution.

mluisbrown commented 11 years ago

Cool! My first ever GitHub pull request got merged :-)