CruCentralCoast / CruiOS

iOS mobile app for Cru Central Coast. NOTE: The app has been decommissioned and this repo has been archived
1 stars 2 forks source link

Profile UI and Functionality #188

Closed tydahlwave closed 6 years ago

tydahlwave commented 6 years ago

Everything is working except for notifications. Also, we will need to use the user's subscribed movements to filter content throughout the app. But that can be in a different PR.

screen shot 2018-09-13 at 6 24 34 pm screen shot 2018-09-13 at 6 24 40 pm screen shot 2018-09-13 at 6 24 45 pm screen shot 2018-09-13 at 6 24 51 pm screen shot 2018-09-13 at 6 34 45 pm screen shot 2018-09-13 at 6 34 40 pm screen shot 2018-09-13 at 6 25 39 pm screen shot 2018-09-13 at 6 26 16 pm screen shot 2018-09-13 at 6 26 24 pm
cbatch commented 6 years ago

instead of setting each color of every navigation interaction, you can just set the global appTint color either from your storyboard file or in application(_:didFinishLaunchingWithOptions:) you can put window?.tintColor = .cruBlue

cbatch commented 6 years ago

one suggestion would be that instead of having choose campus and then a choose movement as a follow up screen, you could just have a choose your movement screen that groups the movements by campus in one tableview. might be simpler and would account for people being involved in multiple movements, even outside of their campus, which I've seen happen

fedorareis commented 6 years ago

Is a search going to be added to the campus page? For now it is probably fine without it but as we expand to more campuses that is definitely going to be needed.

tydahlwave commented 6 years ago

@fedorareis Yes there is a search on the Choose Campus screen if you scroll down. It functions the same as the iMessage search bar

tydahlwave commented 6 years ago

@cbatch True, that may be better. However, the movements would also take up a lot of space on the screen. But it would allow search by campus or movement which would be nice. I'm not going to change the design in this PR to that, but maybe in the future.