-
Notifications
You must be signed in to change notification settings - Fork 477
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
fix: fixed paste on text field error #1524
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
1 Ignored Deployment
|
@anikdhabal is attempting to deploy a commit to the Rowy Team on Vercel. A member of the Team first needs to authorize it. |
Thanks @anikdhabal for the PR - can you share a before and after video to show it working plus showing if paste works fine in other fields as well. |
Yeah, let me share it quickly. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This does not resolve the issues described in the updated issue description, please update.
Primay.rowy-5e51f.Rowy.dev.and.10.more.pages.-.Personal.-.Microsoft_.Edge.2024-01-22.20-48-03.mp4It's working fine now. I've attached the result. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@anikdhabal issue 1 (pasting outside array subtable) is resolved, however issue 2 (pasting into cells into array subtable) persists, please take a look.
Pasting into a short text will throw an error:
Pasting into a number will overwrite the whole array subtable field and crash the page:
Please check and make sure pasting into all supported fields inside array subtable works.
I've updated the code, and now both issues are resolved. |
Upon testing, issue 2 still persists. |
Fixes #1523
/claim #1523