Skip to content

Message Action Menu#1688

Merged
beastoin merged 5 commits intomainfrom
message-action-menu
Jan 16, 2025
Merged

Message Action Menu#1688
beastoin merged 5 commits intomainfrom
message-action-menu

Conversation

@mdmohsin7
Copy link
Copy Markdown
Member

closes #1609

Tap and hold a message, it will show the message action menu, where you can copy, share, and report message

ScreenRecording_01-14-2025.16-07-30_1.1.mp4

@beastoin beastoin merged commit 57d8f4f into main Jan 16, 2025
@beastoin beastoin deleted the message-action-menu branch January 16, 2025 06:36
@kodjima33
Copy link
Copy Markdown
Collaborator

It's better than it was before but it misses the top-1 most used feature: selecting text.

You have to add it and then push.

@beastoin
Copy link
Copy Markdown
Collaborator

@mdmohsin7 man pls add the selecting text option ;)

@mdmohsin7
Copy link
Copy Markdown
Member Author

@beastoin @kodjima33 long tap will not work if text selection (like it was there previously is added). We will have to do it the way chatgpt mobile app does. That is open a new screen with the text for the user to select

screen-20250123-202813.mp4

@kodjima33
Copy link
Copy Markdown
Collaborator

better but can't see selectors

ScreenRecording_02-17-2025.18-31-33_1.MP4

@beastoin
Copy link
Copy Markdown
Collaborator

@mdmohsin7 can you fix it?

@mdmohsin7
Copy link
Copy Markdown
Member Author

I don’t think I understood it correctly. Which selectors exactly? Can you please share a screenshot or something @kodjima33 @beastoin

@beastoin
Copy link
Copy Markdown
Collaborator

the cursor @mdmohsin7

@beastoin
Copy link
Copy Markdown
Collaborator

Screenshot 2025-02-18 at 11 42 11

to tap and select the text

@mdmohsin7
Copy link
Copy Markdown
Member Author

Oh those handles, although we make markdown selectable and not text in kinda a hacky way, can still add those handles using text selection controls @beastoin

@beastoin
Copy link
Copy Markdown
Collaborator

@mdmohsin7 could you fix it sir ?

@mdmohsin7
Copy link
Copy Markdown
Member Author

Yes sir let me, couldn't do it earlier because of the personas work @beastoin

@mdmohsin7
Copy link
Copy Markdown
Member Author

Pls check #1869 @beastoin

@beastoin
Copy link
Copy Markdown
Collaborator

@mdmohsin7
Copy link
Copy Markdown
Member Author

Does it show up on ios?

@beastoin
Copy link
Copy Markdown
Collaborator

@mdmohsin7 thank you for pointing it out / could you pull the main and check it - new PR just got merged 🌚

@mdmohsin7
Copy link
Copy Markdown
Member Author

@mdmohsin7 thank you for pointing it out / could you pull the main and check it - new PR just got merged 🌚

Just tested, it works on iOS as well @beastoin

@beastoin
Copy link
Copy Markdown
Collaborator

with one more line :)

product change logs

  1. the fixes are ready on testflight/internal test 🚀

@mdmohsin7 please check it / @kodjima33 fyi ~

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Option to report ai generated content in chat

3 participants