MortimerGoro / MGSwipeTableCell

An easy to use UITableViewCell subclass that allows to display swippable buttons with a variety of transitions.
MIT License
6.96k stars 1.07k forks source link

duplicate swipe buttons #314

Open hitrain opened 6 years ago

hitrain commented 6 years ago

simulator screen shot - iphone 8 plus - 2018-05-09 at 14 55 42

mail app changed callback YES to NO

MGSwipeButton * flag = [MGSwipeButton buttonWithTitle:@"Flag" backgroundColor:[UIColor colorWithRed:1.0 green:149/255.0 blue:0.05 alpha:1.0] padding:padding callback:^BOOL(MGSwipeTableCell *sender) {

MailData * mail = [me mailForIndexPath:[me.tableView indexPathForCell:sender]];
            mail.flag = !mail.flag;
            [me updateCellIndicactor:mail cell:(MailTableCell*)sender];
            [cell refreshContentView]; //needed to refresh cell contents while swipping
            return NO;
        }];

broken behavior line [cell refreshContentView]; //needed to refresh cell contents while swipping

1.6.6 was fine, 1.6.7 broken

selina commented 6 years ago

Also having this issue. In particular, changing orientation on iPad led to duplication when the buttons were kept swiped. Downgrading to 1.6.6 solved it, but it's not ideal.

sofierPoS commented 6 years ago

Same here. Pls fix....

MortimerGoro commented 6 years ago

It should be fixed in 1.6.8

andres-cianio commented 5 years ago

Confirm, fixed in 1.6.8. Thanks!