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.

Sign up for GitHub

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

Jump to bottom

Long-press message/chat preview is missing #20312

Closed
qoqobolo opened this issue Jun 3, 2024 · 1 comment · Fixed by #20349
Closed

Long-press message/chat preview is missing #20312

qoqobolo opened this issue Jun 3, 2024 · 1 comment · Fixed by #20349
Assignees
@Parveshdhull @jo-mut
Labels
mobile-core
Milestone
2.29.0 Beta

Comments

@qoqobolo
Copy link
Contributor

qoqobolo commented Jun 3, 2024 •
edited
Loading

Bug Report

Problem

We should display a message/chat preview on long press.

Reproduction

1 - Message

  1. Send a message
  2. Long press on the message

2 - Chat

  1. Open the home screen
  2. Long press on the chat

3 - Reactions

  1. Send a message
  2. Add a reaction
  3. Long press on the reaction

Expected behavior

https://www.figma.com/design/7KIYbhoqNGAIFonE0w9TDz/Messages-for-Mobile?node-id=1447-349773&t=Oz48qhdeErsOTl7W-4

https://www.figma.com/design/7KIYbhoqNGAIFonE0w9TDz/Messages-for-Mobile?node-id=1803-400893&t=Oz48qhdeErsOTl7W-4

https://www.figma.com/design/PPWkgOYlZZDxZv5SDGsZVV/Posts-%26-Attachments-for-Mobile?node-id=2869-259157&t=zFLgxEKPDjeSSnGu-4

Screenshot 2024-06-03 at 17 49 05 Screenshot 2024-06-03 at 17 49 24 Screenshot 2024-06-03 at 17 51 51

Actual behavior

Screenshot 2024-06-03 at 17 48 37 Screenshot 2024-06-03 at 17 51 05 Screenshot 2024-06-03 at 17 52 46

Additional Information

  • Status version: RC 2.29
  • Operating System: Android, iOS
@qoqobolo qoqobolo added the bug label Jun 3, 2024
@qoqobolo qoqobolo added this to the 2.29.0 Beta milestone Jun 3, 2024
@jo-mut jo-mut self-assigned this Jun 3, 2024
@cammellos cammellos moved this to In Progress in Status Desktop/Mobile Board Jun 3, 2024
@yqrashawn
Copy link
Contributor

this happened before and was fixed by removing overflow hidden in bottom sheet here https://github.com/status-im/status-mobile/pull/19630/files#diff-3106cb7d6b9fa0b0c3f3a96271519c0b7a076d993ecb87fe34dd5c0dd529bd7bL11

@Parveshdhull Parveshdhull self-assigned this Jun 5, 2024
@Parveshdhull Parveshdhull mentioned this issue Jun 5, 2024
Merged
@github-project-automation github-project-automation bot moved this from In Progress to Done in Status Desktop/Mobile Board Jun 5, 2024
@ilmotta ilmotta removed the bug label Oct 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Parveshdhull Parveshdhull

@jo-mut jo-mut

Labels
mobile-core
Projects
Status Desktop/Mobile Board
Archived in project
Milestone
2.29.0 Beta
Development

Successfully merging a pull request may close this issue.

fix long-press message/chat preview is missing status-im/status-mobile
5 participants
@ilmotta @yqrashawn @Parveshdhull @jo-mut @qoqobolo