Dismiss or pop view controllers accordingly

[aizatomar]

Hello Tim!

Got caught up with Assassin's Creed Valhalla release yesterday! So sorry! I can't help it!

Anyway, here is my PR for improving the dismissing of the view controller in your default done and cancel handler.

I added a check to see if the view controller is embedded in a navigation controller. If it is the root of the navigation controller, dismiss the navigation controller. If it is not, then just pop the view controller.

If the view controller is presented modally without being embedded in a navigation controller, then just dismiss the view controller accordingly.

Note:
I see you have this on line 894:

self.modalTransitionStyle = UIModalTransitionStyleCoverVertical;

I took the liberty of deleting this line because I am not sure why this is in the cancel handler but not in the done handler.

Please let me know if you still want this behavior. And maybe if you want to standardise it for both cancel and done.

[TimOliver]

Hey Aizat! Hahaha very good! I'm trying my hardest to get an Xbox Series X at the moment.

Cool! That looks fine to me! Thanks so much for doing that! :)

[aizatomar]

Good morning and no problem! I'm playing on an Xbox One atm. :)