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.

Sign up for GitHub

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

Jump to bottom

Table Editor widget: pasting #10760

Closed
Tracked by #11183
farmaazon opened this issue Aug 6, 2024 · 1 comment · Fixed by #11332
Closed
Tracked by #11183

Table Editor widget: pasting #10760

farmaazon opened this issue Aug 6, 2024 · 1 comment · Fixed by #11332
Assignees
@farmaazon
Labels
-gui d-hard Difficulty: significant prior knowledge required p-medium Should be completed in the next few sprints x-new-feature Type: new feature request
Milestone
2024-10 Release

Comments

@farmaazon
Copy link
Contributor

Follow up of #10293

The copying data in the widget works exactly the same way as copying from Table Visualization. We should also allow pasting such data to the widget, as well as from external editors.
@farmaazon farmaazon added d-hard Difficulty: significant prior knowledge required p-medium Should be completed in the next few sprints x-new-feature Type: new feature request -gui labels Aug 6, 2024
@farmaazon farmaazon added this to the 2024-08 Release milestone Aug 6, 2024
@github-project-automation github-project-automation bot moved this to ❓New in Issues Board Aug 6, 2024
@farmaazon
Copy link
Contributor Author

farmaazon commented Aug 21, 2024 •
edited
Loading

  • Copying with ctrl+c and pasting with ctrl+v interferes with Graph Editor handlers: the node copy overrides the grid copy, and paste often pastes both to cells and to a new node. Otherwise, CVS data seems to be pasted just fine.
  • Pasting from google sheets seems to work properly.
  • Nice to have: Add simple e2e tests for pasting grids (both to node and to grid) - if it's supported by playwright

@farmaazon farmaazon moved this from ❓New to 📤 Backlog in Issues Board Aug 21, 2024
@AdRiley AdRiley mentioned this issue Sep 26, 2024
Open
@farmaazon farmaazon self-assigned this Oct 14, 2024
@farmaazon farmaazon moved this from 📤 Backlog to 🔧 Implementation in Issues Board Oct 14, 2024
@farmaazon farmaazon mentioned this issue Oct 15, 2024
Merged
2 tasks
@farmaazon farmaazon moved this from 🔧 Implementation to 👁️ Code review in Issues Board Oct 17, 2024
@mergify mergify bot closed this as completed in #11332 Oct 21, 2024
@mergify mergify bot closed this as completed in fa87a18 Oct 21, 2024
@github-project-automation github-project-automation bot moved this from 👁️ Code review to 🟢 Accepted in Issues Board Oct 21, 2024
@github-project-automation github-project-automation bot moved this from 👁️ Code review to 🟢 Accepted in Issues Board Oct 21, 2024
@farmaazon farmaazon moved this from 🟢 Accepted to 🗄️ Archived in Issues Board Oct 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@farmaazon farmaazon

Labels
-gui d-hard Difficulty: significant prior knowledge required p-medium Should be completed in the next few sprints x-new-feature Type: new feature request
Projects
Issues Board
Status: 🗄️ Archived
Milestone
2024-10 Release
Development

Successfully merging a pull request may close this issue.

Copying and pasting in Grid fixes. enso-org/enso
2 participants
@AdRiley @farmaazon