Skip to content

Conversation

soKaradag
Copy link
Collaborator

No description provided.

@gunesmert
Copy link
Collaborator

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.
@gunesmert gunesmert changed the title Pano/Swift/SwiftUI Pano ~ UI revamp / rebuild Jul 25, 2023
@gunesmert
Copy link
Collaborator

First off, great start @soKaradag!! I will be mentioning some nice to haves before having a look at the PR:

  • It would be nice to have a descriptive title to the PR. In our case I am switching it from Pano/Swift/SwiftUI to Pano ~ Initial UI revamp / rebuild
  • It would be nice to have a cool description about the content of this PR. All in all, this is your PR and you need to showcase what you have done / changed / applied to the outside world. It is really important for outsiders to see what happened on a PR without looking at the code itself. Description can contain stuff like:
    • List of changes (preferably with a small explanation and link to the related issue)
    • A screenshot or a video of the change
  • Related to ☝️ , it would be nice to see the issues updated according to the changes of this PR. I can see that our issues section is not fully organized yet but we should be able to track the progress on each of them. So, I would suggest you to update the related issues once we merge this PR with your progress.

@gunesmert gunesmert self-assigned this Jul 25, 2023
@gunesmert gunesmert added the enhancement New feature or request label Jul 25, 2023
Copy link
Collaborator

@gunesmert gunesmert left a 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 💪 !!

Copy link
Collaborator Author

@soKaradag soKaradag left a 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.

Copy link
Collaborator Author

@soKaradag soKaradag left a 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.
@gunesmert gunesmert self-requested a review September 14, 2023 21:02
@gunesmert gunesmert merged commit 3795706 into kamp-us:main Sep 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants