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

Add drag to reveal timestamps #696

Closed
saulmc opened this issue Sep 9, 2024 · 4 comments · Fixed by #749
Closed

Add drag to reveal timestamps #696

saulmc opened this issue Sep 9, 2024 · 4 comments · Fixed by #749
Assignees
Labels
enhancement New feature or request

Comments

@saulmc
Copy link
Member

saulmc commented Sep 9, 2024

Dragging to the left anywhere inside a conversation should reveal message timestamps to the right.

When dragging to the left, messages from me should translate left accordingly, but messages from others should stay in place. This interaction can be seen in iMessage.

@saulmc saulmc self-assigned this Sep 9, 2024
@saulmc saulmc added the enhancement New feature or request label Sep 13, 2024
@saulmc saulmc assigned lourou and unassigned saulmc Sep 13, 2024
@lourou
Copy link
Member

lourou commented Sep 17, 2024

@saulmc I've implemented the tap-to-reveal timestamps feature. Let me know what you think!

Time displays are localized based on the 12/24-hour formats depending on the phone's locale.

@saulmc
Copy link
Member Author

saulmc commented Sep 17, 2024

Thanks @lourou ! Could you please modify the animation so that the container for the timestamp expands vertically, not just the timestamp text? This will make the message list feel less jumpy in reaction to the timestamp reveal.

@lourou
Copy link
Member

lourou commented Sep 18, 2024

Yes @saulmc this is now smooth both animating in and out!

@saulmc
Copy link
Member Author

saulmc commented Sep 18, 2024

Beautiful! Thanks!

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 a pull request may close this issue.

2 participants