Skip to content
This repository has been archived by the owner on Feb 21, 2019. It is now read-only.

Attempting to paste over multiple lines of code results in error #26

Open
Nantris opened this issue Nov 12, 2017 · 7 comments
Open

Attempting to paste over multiple lines of code results in error #26

Nantris opened this issue Nov 12, 2017 · 7 comments
Labels

Comments

@Nantris
Copy link

Nantris commented Nov 12, 2017

If you highlight a line and paste over it, it works perfectly.

If you highlight 2 or more lines and paste over them with text from anywhere besides the nodes you are pasting over, you'll get Uncaught Error: Could not find a node with key "###"

If you copy identical text or non-HTML (plaintext) over the nodes, it generally seems to work.

If you use Ctrl+A to select all contents of the codebox and paste, it seems to fail universally, even if you paste plaintext.

@Nantris
Copy link
Author

Nantris commented Nov 16, 2017

Some quirks to note:

  1. This only occurs when highlighting multiple lines IN FULL. If any line is partially highlighted, pasting works.
  2. When highlighting multiple lines with the mouse cursor, it only occurs if you select in reverse
  3. When highlighting multiple lines with the keyboard, it only occurs if you select forward (not in reverse)

By reverse I mean where the anchor node is after the focus node.

@Soreine Soreine added the bug label Nov 19, 2017
@Soreine
Copy link
Contributor

Soreine commented Nov 19, 2017

Strange :O
I'll try to reproduce this...

@Nantris
Copy link
Author

Nantris commented Nov 19, 2017

@Soreine thanks very much. You guys are the best. I'll try to submit a PR for the other issues I reported later this week.

@Soreine
Copy link
Contributor

Soreine commented Nov 27, 2017

Just to keep you informed, I don't have the time to dive in this right now...

@Nantris
Copy link
Author

Nantris commented Nov 28, 2017

No worries. I figured you had more important things to work on on your end. I definitely appreciate the update. Thanks very much for the great work.

@Soreine
Copy link
Contributor

Soreine commented Dec 27, 2017

Updating to [email protected] fixed a few of these issues, but some remains.

Triple-click on a code line and paste => error

Slate issue ianstormtaylor/slate#1493

@Nantris
Copy link
Author

Nantris commented Dec 27, 2017

@Soreine you rock! Thanks very much for the hard work and for the update!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

2 participants