Skip to content
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

Profile screen in SwiftUI #963

Merged
merged 37 commits into from
Nov 25, 2022
Merged

Profile screen in SwiftUI #963

merged 37 commits into from
Nov 25, 2022

Conversation

martindsq
Copy link
Member

@martindsq martindsq commented Nov 16, 2022

Closes #681, #976, #977 and #961

@mplorentz
Copy link
Member

Going to post some issues I have seen in TestFlight here. Will put them in separate comments as maybe that will help track which ones have been fixed. I'll put checkboxes on them too in case that is helpful? I don't think we get the "Resolve conversation" button unless I comment on a specific line of code.

@mplorentz
Copy link
Member

mplorentz commented Nov 16, 2022

  • The ellipsis button is working for me on any posts after I scroll down a bit.

@mplorentz
Copy link
Member

mplorentz commented Nov 16, 2022

  • The Read More button has no pressed state.

@mplorentz
Copy link
Member

mplorentz commented Nov 16, 2022

  • hashtag detection isn't working for camel case hashtags

@mplorentz
Copy link
Member

mplorentz commented Nov 16, 2022

  • The ignored list of people has the title "Blocked". I think it should be "Ignored". (maybe this is a problem in the ASV as well. We can open a separate ticket for this if so)

@mplorentz
Copy link
Member

mplorentz commented Nov 16, 2022

  • opening profiles from a search on the discover tab opens the old profile screen

@mplorentz
Copy link
Member

mplorentz commented Nov 16, 2022

  • This is more of a design question maybe (CC @Chardot) but I wish the header would collapse down smaller. On my 12 mini I feel like I am losing about ¼ of my screen real estate to the header. Maybe we could hide the following button in the collapsed state and we could shrink the profile picture. Manyverse has a nice animation for the header collapse we could emulate.
    IMG_CFC9D1CBC149-1

@martindsq
Copy link
Member Author

  • The ellipsis button is working for me on any posts after I scroll down a bit.

I couldn't reproduce this. Can you check if it you still see it in today's build in Testlight? If you still do, let me know.

@mplorentz
Copy link
Member

I couldn't reproduce this. Can you check if it you still see it in today's build in Testlight? If you still do, let me know.

Just verifies that I am still seeing it. Here's a video of my exact steps. If you still can't reproduce then I can check out the branch and do some debugging.

RPReplay_Final1668703525.MP4

@martindsq
Copy link
Member Author

Sorry @mplorentz but that's not the build I sent today. Its still processing I think.

@Chardot
Copy link

Chardot commented Nov 17, 2022

  • The Read More button has no pressed state.

@martindsq For the pressed state we can invert the colors of the button in both light and dark mode. Here's a preview:

Screenshot 2022-11-17 at 18 01 23

@Chardot
Copy link

Chardot commented Nov 17, 2022

  • This is more of a design question maybe (CC @Chardot) but I wish the header would collapse down smaller. On my 12 mini I feel like I am losing about ¼ of my screen real estate to the header. Maybe we could hide the following button in the collapsed state and we could shrink the profile picture. Manyverse has a nice animation for the header collapse we could emulate.
    IMG_CFC9D1CBC149-1

This is a very good suggestion, @mplorentz . Here's a smaller collapsed version of the profile card:

Screenshot 2022-11-17 at 17 50 51

@Chardot
Copy link

Chardot commented Nov 17, 2022

@martindsq Here's an animated prorotype for the collapsing of the profile card:

Screen.Recording.2022-11-17.at.18.34.37.mov

@mplorentz
Copy link
Member

I think we should probably limit the user's name to one line? Although it would feel bad to truncate people who just have long names. Maybe we need some guidance on how to make long names look good @Chardot?

@mplorentz
Copy link
Member

The nav bar is transparent on the latest commit in my 14 pro simulator.

@mplorentz
Copy link
Member

Screenshot 2022-11-21 at 4 04 52 PM

@Chardot I think we could use some kind of placeholder for when a user isn't following/followed by/ignoring anyone?

@martindsq
Copy link
Member Author

@mplorentz

I think we should probably limit the user's name to one line? Although it would feel bad to truncate people who just have long names. Maybe we need some guidance on how to make long names look good @Chardot?

I went ahead and made it a two line limit label (because we can), and instead of using the full identity when the name is not available, it now uses the short identifier

The nav bar is transparent on the latest commit in my 14 pro simulator.

Fixed this

Screenshot 2022-11-21 at 4 04 52 PM

@Chardot I think we could use some kind of placeholder for when a user isn't following/followed by/ignoring anyone?

When implementing the profile screen I asked this same question and he preferred to show it as it is.

@martindsq martindsq merged commit 8710049 into main Nov 25, 2022
@martindsq martindsq deleted the feature/about_view_swiftui branch November 25, 2022 11:59
@Chardot
Copy link

Chardot commented Nov 30, 2022

Screenshot 2022-11-21 at 4 04 52 PM

@Chardot I think we could use some kind of placeholder for when a user isn't following/followed by/ignoring anyone?

That's a very good suggestion. What about something like this?

Screenshot 2022-11-30 at 18 04 06

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants