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

feat: Move Pinned Conversation Indicator #922

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

alexrisch
Copy link
Collaborator

@alexrisch alexrisch commented Oct 4, 2024

Moved Pinned Conversation indicator to left of title Removed Avatar Indicator
Cleaned up style of indicator to be more reusable
Closes #920

Simulator Screenshot - iPhone 15 Plus - 2024-10-04 at 09 32 35
Simulator Screenshot - iPhone 15 Plus - 2024-10-04 at 09 32 28
Simulator Screenshot - iPhone 15 Plus - 2024-10-04 at 09 31 23
Simulator Screenshot - iPhone 15 Plus - 2024-10-04 at 09 31 12

Moved Pinned Conversation indicator to left of title
Removed Avatar Indicator
Cleaned up style of indicator to be more reusable
@alexrisch alexrisch requested a review from a team as a code owner October 4, 2024 13:40
@saulmc
Copy link
Member

saulmc commented Oct 4, 2024

Let's tag @covrter for visibility on these smaller visual adjustments going forward

@covrter
Copy link
Collaborator

covrter commented Oct 4, 2024

Great shout, thanks y'all

@alexrisch
Copy link
Collaborator Author

@saulmc @covrter This is the work noted in #729
Should I both at once, fine to do incrementally or do we want to hold off on this for now?

@saulmc
Copy link
Member

saulmc commented Oct 4, 2024

Fine to do incrementally imo!

<Text style={styles.text}>{title}</Text>
<View style={styles.textContainer}>
{showUnread && (
<View>
Copy link
Collaborator

@thierryskoda thierryskoda Oct 5, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why wrapping with this View?

Copy link
Collaborator

@thierryskoda thierryskoda left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same comments as I did in the other one. I would just make sure that we do those because it's P0 IMO.

I'll get the base of the design-system next week for sure so we can start using it

@alexrisch
Copy link
Collaborator Author

Will hold onto this PR for a bit while we get the design system in place

@alexrisch alexrisch marked this pull request as draft October 7, 2024 16:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Show an unread indicator badge to the left of the conversation name
4 participants