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

Close Chat and Clear History flows updated to align with mobile app in terms of functionality and terminology #13393

Closed
benjthayer opened this issue Feb 1, 2024 · 2 comments · Fixed by #14118

Comments

@benjthayer
Copy link

benjthayer commented Feb 1, 2024

The Close Chat and Clear History context menu items for 1:1 chat have been updated to align with mobile.

Screenshot 2024-02-01 at 9 12 25 PM

In addition the confirmation dialogs for each action have also been updated to align with mobile:

Close chat:

Screenshot 2024-02-01 at 9 14 08 PM

Once the chat is closed, the chat disappears from the user's Chat list and the next chat down in the list is selected. If the user is closing their only chat, the empty state for the Messaging app is shown.

--

Clear chat history:
Note that there are 2 checkboxes:
Delete all messages on my side- mandatory (on and disabled)
Delete my messages for [counterparty name] - optional - if checked, this will delete all of the actioning user's messages from the chat on the counterparty side.

Screenshot 2024-02-01 at 9 14 33 PM

Figma:
https://www.figma.com/file/Mr3rqxxgKJ2zMQ06UAKiWL/%F0%9F%92%AC-Chat%E2%8E%9CDesktop?type=design&node-id=24942-215153&mode=design&t=GWTxxV5I2KBpua8E-4

@iurimatias iurimatias added this to the 2.29.0 Beta milestone Feb 14, 2024
@alexandraB99 alexandraB99 self-assigned this Mar 22, 2024
alexandraB99 added a commit to status-im/status-go that referenced this issue Apr 5, 2024
alexandraB99 added a commit to status-im/status-go that referenced this issue Apr 15, 2024
@noeliaSD noeliaSD self-assigned this Apr 16, 2024
@noeliaSD
Copy link
Contributor

Assigned to myself to better analyse

@noeliaSD noeliaSD removed their assignment May 7, 2024
@noeliaSD noeliaSD modified the milestones: 2.29.0 Beta, 2.31.0 Beta May 7, 2024
@noeliaSD
Copy link
Contributor

noeliaSD commented May 7, 2024

It seems there's an open pr that needs to be reevaluated.
Moved to 2.31 since in 2.30, the ui-team will be focused on wallet.

@noeliaSD noeliaSD modified the milestones: 2.31.0 Beta, 2.32.0 Beta Jun 26, 2024
@caybro caybro self-assigned this Sep 3, 2024
caybro pushed a commit to status-im/status-go that referenced this issue Sep 4, 2024
caybro pushed a commit to status-im/status-go that referenced this issue Sep 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

6 participants