Pasted content is placed at the beginning of the message and not where the cursor is positioned and undo (CTR+Z) is not working

[isagoico]

If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!

---

Action Performed:

1. Log in to e.cash and navigate to a conversation
2. Send a message
3. After the message has sent, copy it
4. In the compose box, enter some text
5. With the cursor at the end of the input, paste the message
6. Press the shortcut for undo (CTRL+Z)

Expected Result:

Pasted content should be displayed were the cursor was positioned, in this case at the end of the message. CTRL+Z should undo the last action that was the pasted content.

Actual Result:

Pasted content is displayed at the start of the message and CRTL+Z does not remove the pasted content.

Workaround:

Can the user still use Expensify without this being fixed? Have you informed them of the workaround?

Platform:

Where is this issue occurring?

Web ✔️
iOS
Android
Desktop App ✔️
Mobile Web

Version Number: 1.0.79-0

Logs: https://stackoverflow.com/c/expensify/questions/4856

Notes/Photos/Videos:

Expensify/Expensify Issue URL:

View all open jobs on Upwork

---

From @alex-mechler https://expensify.slack.com/archives/C01GTK53T8Q/p1626366219328100

When pasting content from another message on Desktop, the pasted content is always placed at the beginning of the compose, regardless of where the cursor is. Reproduction steps in the thread

[MelvinBot]

Triggered auto assignment to @mountiny (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

[mountiny]

This seems like it can be worked on by a contributor! Adding External label then.

[MelvinBot]

Triggered auto assignment to @trjExpensify (External), see https://stackoverflow.com/c/expensify/questions/8582 for more details.

[mountiny]

But I am going to keep myself assigned and look into to once I will have time if noone will make proposal to solve it before that.

[rdjuric]

@mountiny This looks like a regression from #4009

ccing @parasharrajat, looks like this is the problematic part https://github.com/Expensify/Expensify.cash/blob/bb02d771dc51ff371d32da27e150a36f495124e0/src/components/TextInputFocusable/index.js#L254-L257

[parasharrajat]

Thanks, @rdjuric. I am looking at it already. There is one thing that is bothering me at ATM how ctrl+z will work. What is the best thing to do here? Should we really need to override this via keyboard event?