-
Notifications
You must be signed in to change notification settings - Fork 0
Pano ~ UI revamp / rebuild #13
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Still not sure about using customTabbar but in TabView can't use Model Navigation. Open to suggestions for this problem.
Listing current user's posts in profile.
Thanks for the contribution @soKaradag ! Will have look asap. |
Added Preview Toggle. User can chose is preview gonna be visible.
if user login with account shows search and profile view. If not show Login/Register view.
First off, great start @soKaradag!! I will be mentioning some nice to haves before having a look at the PR:
|
Pano.xcodeproj/project.xcworkspace/xcuserdata/serdar.xcuserdatad/UserInterfaceState.xcuserstate
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It looks good but I think there is some room for improvements. Added some comments and suggestions for some required changes. Let's do them before merging this PR.
In addition to that, UI looks great. Of course there are still some room for improvements but what you did is not the final implementation and things will change a lot on the go. Keep up with the good work 💪 !!
Now tabbar code more clean with enums and switch case.
Now changes on userDefaults are controlling in same place.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed most of the problems.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Necessary changes have been made.
Now Tabbar enum name changed as TabType and has become more efficient.
Changed Settings view navigation to NavigationLink instead of Sheet.
No description provided.