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

[data grid] Copy/paste places unnecessary quotations when copied text contains a quotation #12456

Closed
louisaaron opened this issue Mar 15, 2024 · 11 comments · Fixed by #12722
Closed
Assignees
Labels
bug 🐛 Something doesn't work component: data grid This is the name of the generic UI component, not the React module! feature: Clipboard Related to clipboard copy or paste functionalities support: pro standard Support request from a Pro standard plan user. https://mui.com/legal/technical-support-sla/

Comments

@louisaaron
Copy link

louisaaron commented Mar 15, 2024

Steps to reproduce

Link to live example: (required)

Steps:

  1. Type content into a cell which includes a quotation mark. For example, Test " Text
  2. Copy it
  3. Paste it into another cell

Current behavior

Extraneous quotation marks are added

Expected behavior

Text should be pasted as it was copied

Context

Trying to copy and paste cells with text which contain a quotation mark

Your environment

npx @mui/envinfo
  Don't forget to mention which browser you used.
  Output from `npx @mui/envinfo` goes here.

Search keywords: quotation
Order ID: 85118

@louisaaron louisaaron added the status: waiting for maintainer These issues haven't been looked at yet by a maintainer label Mar 15, 2024
@michelengelen
Copy link
Member

Hey @louisaaron
thanks for raising this issue. Seems like the pasting operation does some unnecessary parsing there. I will add this to our board!

@michelengelen michelengelen added bug 🐛 Something doesn't work component: data grid This is the name of the generic UI component, not the React module! feature: Clipboard Related to clipboard copy or paste functionalities and removed status: waiting for maintainer These issues haven't been looked at yet by a maintainer labels Mar 15, 2024
@michelengelen michelengelen changed the title Copy/paste places unnecessary quotations when copied text contains a quotation [data grid] Copy/paste places unnecessary quotations when copied text contains a quotation Mar 15, 2024
@michelengelen michelengelen added the support: pro standard Support request from a Pro standard plan user. https://mui.com/legal/technical-support-sla/ label Mar 15, 2024
@sai6855
Copy link
Contributor

sai6855 commented Mar 15, 2024

@michelengelen I've opened #12357 to fix #11945 but looking at PR implementation, I think it also fixed this issue as well

@louisaaron
Copy link
Author

@michelengelen unfortunately this did not solve the problem. Do you have a timeline for a fix?

Thank you!

@louisaaron
Copy link
Author

@michelengelen @sai6855 just following up here to make sure this is in some form of queue. Could you please confirm that you're aware of this bug? Thank you!

@michelengelen
Copy link
Member

@michelengelen just following up here to make sure this is in some form of queue. Could you please confirm that you're aware of this bug? Thank you!

Sry about the delay ... I simply forgot to add it to the board, so thanks for the follow-up! We will have a look in our next grooming session!

@louisaaron
Copy link
Author

Awesome thank you @michelengelen !

@cherniavskii
Copy link
Member

Hi @louisaaron
I've opened #12722 with a fix for the current stable version (v7).
Do you need this backported for the v6 as well?

@louisaaron
Copy link
Author

Thanks @cherniavskii ! It's not the end of the world if not, but we're still on 6 currently so it would be helpful! Let me konw what you decide. thanks again!

Copy link

⚠️ This issue has been closed. If you have a similar problem but not exactly the same, please open a new issue.
Now, if you have additional information related to this issue or things that could help future readers, feel free to leave a comment.

@louisaaron: How did we do? Your experience with our support team matters to us. If you have a moment, please share your thoughts in this short Support Satisfaction survey.

@cherniavskii
Copy link
Member

@louisaaron I'll cherry-pick it to v6 👍🏻

@louisaaron
Copy link
Author

Thanks you @cherniavskii !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🐛 Something doesn't work component: data grid This is the name of the generic UI component, not the React module! feature: Clipboard Related to clipboard copy or paste functionalities support: pro standard Support request from a Pro standard plan user. https://mui.com/legal/technical-support-sla/
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants