OneBusAway / onebusaway-ios

OneBusAway for iOS, written in Swift.
Other
76 stars 32 forks source link

Split contents of MapFloatingPanelController into separate view controllers #661

Closed ualch9 closed 1 year ago

ualch9 commented 1 year ago

I suspect the layout system in OBAListView is partially responsible for #660. As a hotfix, I am replacing the Recent Stops list view with a standard UICollectionView. The current search view controller was more complicated to replace, so that will stay as an OBAListView for now. To facilitate the UICollectionView and OBAListView, the two states are now separate view controllers, and MapFloatingPanelController will facilitate the view controller's lifecycles.

MapFloatingPanelController is now spaghetti, but I want to collect crash data on whether the problem is really with OBAListView's layout before cleaning up the logic.

Also, some opportunistic fixes too.