Properly deselect row upon view re-appearance

[pkluz]

Currently the cell selection tends to bug when the user interactively dismisses an acknowledgement controller. It either stays or lags behind. Fixed by dismissing it manually on view appearance.

[vtourraine]

Yes, I’ve noticed that a lot with iOS 7. I’ve always suspected it’s some kind of bug in UIKit, so I’d rather try to avoid this kind of manual fix.

Oh well. It’s still here with iOS 7.1, so I guess we’ll have to do things ourselves. Thanks for the PR. And if someone can find a more elegant solution, you know what to do.

[pkluz]

This is in fact not a bug but a feature ;-)

A UITableViewController as a matter of fact has the property "clearsSelectionOnViewWillAppear" but in some cases this is not desired so you can turn it off. Now if you manage the table view yourself, this is entirely up to you. Think of a split view controller on the iPad - you most likely wouldn't want it deselecting automatically in most cases, because the selection is always present.

So it's in fact not a work around if you felt that way but a necessary management of the tableview if you're not using the UITableViewController but decide to do it manually :)

[vtourraine]

Well, I’m not so sure about that. This is a standard UITableViewController, and the clearsSelectionOnViewWillAppear property default value is YES. So we shouldn’t have to manually deselect the cell. And as a matter of fact, it does usually deselect itself when you hit the “back” button from the top view controller. The problem is that sometimes it doesn’t, especially when you use the interactive pop gesture, I’ve noticed.

So I’d still call that a bug, but maybe I’m wrong. Anyway, thanks to you we now have the expected result, consistently. Hooray.