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

Copying cells from the table viz on a mac keyboard shortcut #11141

Closed
AdRiley opened this issue Sep 20, 2024 · 3 comments · Fixed by #11332
Closed

Copying cells from the table viz on a mac keyboard shortcut #11141

AdRiley opened this issue Sep 20, 2024 · 3 comments · Fixed by #11332
Assignees
Labels
--bug Type: bug -gui

Comments

@AdRiley
Copy link
Member

AdRiley commented Sep 20, 2024

Copying cells from the table viz on a mac keyboard shortcut should be command-C not Control-C.

Same for other keyboard shortcuts

image
@AdRiley AdRiley added --bug Type: bug -gui labels Sep 20, 2024
@AdRiley AdRiley added this to the 2024-09 Release milestone Sep 20, 2024
@farmaazon farmaazon moved this from ❓New to 📤 Backlog in Issues Board Oct 14, 2024
@farmaazon farmaazon moved this from 📤 Backlog to 🔧 Implementation in Issues Board Oct 14, 2024
@farmaazon farmaazon self-assigned this Oct 15, 2024
@farmaazon farmaazon moved this from 🔧 Implementation to 👁️ Code review in Issues Board Oct 17, 2024
@enso-bot
Copy link

enso-bot bot commented Oct 18, 2024

Adam Obuchowicz reports a new STANDUP for the last Tuesday (2024-10-15):

Progress: Replace default copypaste handlers from AgGrid with our own, configurable. Attach the events properly, so it doesn't interfere anymore with GraphEditor's handler. Updated the menu. It should be finished by 2024-10-21.

Next Day: Next day I will be working on the same task. Code cleanup and tests.

@enso-bot
Copy link

enso-bot bot commented Oct 18, 2024

Adam Obuchowicz reports a new STANDUP for the last Wednesday (2024-10-16):

Progress: During tests discovered, that we handle paste wrongly in case of data being bigger than already created table. This, and the fact that pasting multiple cells made many AST transactions forced me to implement custom paste handler, pasting directly to HTML. Added unit and e2e tests. It should be finished by 2024-10-21.

Next Day: Next day I will be working on the same task. Last testing and create a PR.

@enso-bot
Copy link

enso-bot bot commented Oct 18, 2024

Adam Obuchowicz reports a new STANDUP for yesterday (2024-10-17):

Progress: Created a PR with copying and paste. Tried to quickly unify linters for dashboard and GUI, but it wasn't so simple: lints requiring type checking does work in vue files for some reason, and even smaller set extended the linting time greatly. It should be finished by 2024-10-21.

Next Day: Next day I will be working on the #10862 task. Start next issue.

@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
@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
Labels
--bug Type: bug -gui
Projects
Status: 🗄️ Archived
Development

Successfully merging a pull request may close this issue.

2 participants