Jasonette / JASONETTE-iOS

📡 Native App over HTTP, on iOS
https://www.jasonette.com
MIT License
5.27k stars 352 forks source link

No exception handling for tabbars when modal #284

Closed gliechtenstein closed 6 years ago

gliechtenstein commented 6 years ago

This fixes a tab bar bug where:

  1. Launching a view without a tabbar from a view with a tabbar would result in an empty tabbar at the bottom
  2. When coming back from the previous situation, the original view's tabbars are gone

This was caused by some of the edge case exception handling logic which was intended to make sure the transition is smooth. This makes things like tabbar to full screen push transition, tab1 to tab2 transition, etc. smoother, but introduced complexity when dealing with modals.

But modals are always drawn on a clean slate canvas (we don't have to worry about complex handling such as switching from tab 1 to tab 2 within the same screen) therefore we can skip the edge case exception handling altogether.

That's what I did here. It immediately fixes the issues brought up here https://github.com/Jasonette/JASONETTE-iOS/issues/283 but I expect this to fix many other weird issues that existed when transitioning modally between a view with a tabbar and one without .