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

🐛 Bug: Cursor movements blocked during view rename #4686

Closed
1 of 8 tasks
dstala opened this issue Dec 22, 2022 · 1 comment
Closed
1 of 8 tasks

🐛 Bug: Cursor movements blocked during view rename #4686

dstala opened this issue Dec 22, 2022 · 1 comment
Assignees
Labels
🐛 Type: Bug Something is broken or incorrect unexpectedly.

Comments

@dstala
Copy link
Member

dstala commented Dec 22, 2022

Please confirm if bug report does NOT exists already ?

  • I confirm there is no existing issue for this

Steps to reproduce ?

double click on view name to toggle into edit mode

  • move cursor
    • left arrow & right arrow should move one text to left or right
    • up arrow & down arrow should move to beginning and end of text
  • drag mouse to select text
  • double click to select text, use cmd+c to copy; paste contents - throws error
Cannot read properties of undefined (reading `uidt`)
  • lengthy view name needs to be truncated in toolbar view

Screenshot 2022-12-22 at 3 51 44 PM

  • use of delete key in view name edit mode throws error; backspace works fine
Grid.vue:405 Uncaught (in promise) TypeError: Cannot read properties of undefined (reading 'uidt')
    at clearCell (Grid.vue:405:3)
    at HTMLDocument.handleKeyDown (index.ts:217:9)

Desired Behavior

expect keyboard operations to work as desired

Project Details

Node: v16.14.0
Arch: arm64
Platform: darwin
Docker: false
RootDB: sqlite3
PackageVersion: 0.100.2

Attachements

No response

@dstala dstala added the 🐛 Type: Bug Something is broken or incorrect unexpectedly. label Dec 22, 2022
pranavxc added a commit that referenced this issue Dec 24, 2022
re #4686 - lengthy view name needs to be truncated in toolbar view

Signed-off-by: Pranav C <pranavxc@gmail.com>
pranavxc added a commit that referenced this issue Dec 24, 2022
re #4686

Signed-off-by: Pranav C <pranavxc@gmail.com>
pranavxc added a commit that referenced this issue Dec 26, 2022
re #4686

Signed-off-by: Pranav C <pranavxc@gmail.com>
pranavxc added a commit that referenced this issue Dec 26, 2022
re #4686 - lengthy view name needs to be truncated in toolbar view

Signed-off-by: Pranav C <pranavxc@gmail.com>
pranavxc added a commit that referenced this issue Dec 26, 2022
re #4686

Signed-off-by: Pranav C <pranavxc@gmail.com>
@wingkwong wingkwong added the 👀 Status: In Review The PR is currently being verified. label Dec 27, 2022
@wingkwong wingkwong added this to the Next Release: 0.101.0 milestone Dec 27, 2022
@pranavxc pranavxc added 🚀 Status: Ready for Next Release and removed 👀 Status: In Review The PR is currently being verified. labels Dec 27, 2022
@github-actions
Copy link
Contributor

Fixed in v0.101.0-beta.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 Type: Bug Something is broken or incorrect unexpectedly.
Projects
Development

No branches or pull requests

3 participants