Skip to content

Delete Key Fix#6

Open
AlexArendsen wants to merge 2 commits intomasterfrom
delete-key-fix
Open

Delete Key Fix#6
AlexArendsen wants to merge 2 commits intomasterfrom
delete-key-fix

Conversation

@AlexArendsen
Copy link
Owner

Delete key now works. Both the backspace and delete keys now take a higher priority, triggering a content disparity check each time they are pressed. The changes in #5 were required for the fix, and so they are included in this PR.

Users can now move through inserted text using the arrow keys. Adding text to
inserted texts invalidates the inserted text, and pressing backspace inside an
inserted text deletes it altogether. Pre-existing inserted texts must now be
initialized with a `data-atwho-chosen-value` attribute whose value must equal
the contents of the inserted text (this should be later automated).
Delete and backspace keys now trigger more general change detection that works
for both keys.
@AlexArendsen
Copy link
Owner Author

This fixes #4.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant