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

Shift-Delete doesn't copy to clipboard #9461

Closed
lnicola opened this issue Apr 13, 2019 · 9 comments
Closed

Shift-Delete doesn't copy to clipboard #9461

lnicola opened this issue Apr 13, 2019 · 9 comments

Comments

@lnicola
Copy link

lnicola commented Apr 13, 2019

Description

Shift-Delete should traditionally cut the text, putting it into the clipboard, same as Ctrl-X / Cmd-X. Ctrl-X works fine.

Steps to reproduce

  • type some text, copy it to clipboard, delete it
  • type something else, select it, cut with Shift-Del
  • paste

Log: not sent

Version information

  • Platform: web
  • Browser: Firefox Nightly
  • OS: Arch Linux
  • URL: self-hosted, 0.99.2-3
@jryans
Copy link
Collaborator

jryans commented Apr 15, 2019

@lnicola Where are you typing? In the message composer?

@jryans jryans added the X-Needs-Info This issue is blocked awaiting information from the reporter label Apr 15, 2019
@lnicola
Copy link
Author

lnicola commented Apr 15, 2019

@jryans In this pane:

image

@jryans jryans added type:composer and removed X-Needs-Info This issue is blocked awaiting information from the reporter labels Apr 15, 2019
@jryans
Copy link
Collaborator

jryans commented Apr 15, 2019

Thanks! My guess is that this is another variant of #7665 and will be fixed by #7697.

@lnicola
Copy link
Author

lnicola commented Apr 16, 2019

I'm confused. That sounds like a workaround, not a fix. Can't the existing editor handle Shift-Del just like it handles Ctrl-X? It's a basic editing shortcut, not something complex like an input method for a non-Latin script.

@t3chguy
Copy link
Member

t3chguy commented Apr 16, 2019

@lnicola could you try it here: https://www.slatejs.org/#/rich-text

If it doesn't work there then it is an issue upstream in the composer we use (slatejs) and not something we can work around without implementing something like #7697

@lnicola
Copy link
Author

lnicola commented Apr 16, 2019

Yeah, I just tried it, and it doesn't work. It's an upstream bug.

@lnicola
Copy link
Author

lnicola commented Apr 16, 2019

Filed ianstormtaylor/slate#2694.

@lampholder
Copy link
Member

related: #6531

@lampholder lampholder added the P2 label Apr 30, 2019
@lnicola
Copy link
Author

lnicola commented Oct 25, 2019

Fixed in the new composer (1.5.0?).

@lnicola lnicola closed this as completed Oct 25, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants