firepad icon indicating copy to clipboard operation
firepad copied to clipboard

Undo is removing the whole multiline code instead of current line.

Open mbsyaswanth opened this issue 5 years ago • 2 comments

Version info

Firebase: 7.13.2 npm package

Firepad: 1.5.10

Ace: react-ace - 8.0.0

Test case

http://plnkr.co/edit/pB0OPUMsaDgeQKsL

Steps to reproduce

Guess this is the same with any kind of firepad integration with ace. Use the above link to test this behaviour.

Expected behavior

when i enter multiple lines of code, line by line and press ctrl+z(undo) . I expect the undo function to remove only the code that i am entering in current line. you can test this in expected behaviour in ace editor demo - http://securingsincity.github.io/react-ace/

Actual behavior

when i enter multiple lines of code, line by line and press ctrl+z(undo) , it removes all the code in the editor. use the link provided in testcase section above

mbsyaswanth avatar Apr 08 '20 10:04 mbsyaswanth

@mbsyaswanth can you explain what you mean by "enter multiple lines of code"? Do you mean pasting multiple lines in at once or do you mean typing a few lines of code manually?

samtstern avatar Apr 09 '20 13:04 samtstern

@mbsyaswanth can you explain what you mean by "enter multiple lines of code"? Do you mean pasting multiple lines in at once or do you mean typing a few lines of code manually?

I'm sure he means typing in a few lines of code manually. I can reproduce this issue.

adamjimenez avatar May 20 '21 07:05 adamjimenez