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

Autosave deletes contents of block if block has focus #2300

Closed
anna-harrison opened this issue Aug 9, 2017 · 3 comments · Fixed by #2319
Closed

Autosave deletes contents of block if block has focus #2300

anna-harrison opened this issue Aug 9, 2017 · 3 comments · Fixed by #2319
Assignees
Labels
[Priority] High Used to indicate top priority items that need quick attention [Type] Bug An existing feature does not function as intended
Milestone

Comments

@anna-harrison
Copy link

If you are typing in a block, and autosave fires, contents of the block are deleted
Seems to happen only on the first firing of autosave in a new document

autosavebug9aug2017

@anna-harrison anna-harrison added the [Type] Bug An existing feature does not function as intended label Aug 9, 2017
@aduth aduth self-assigned this Aug 9, 2017
@aduth aduth added this to the Beta 0.8.0 milestone Aug 9, 2017
@aduth
Copy link
Member

aduth commented Aug 9, 2017

Suspected that regression was introduced in #1996.

Self-assigned.

@aduth aduth added the [Priority] High Used to indicate top priority items that need quick attention label Aug 9, 2017
@aduth
Copy link
Member

aduth commented Aug 9, 2017

Bisect reveals da5cbe7 as problematic commit.

@ellatrix
Copy link
Member

ellatrix commented Aug 10, 2017

Also ran into this just now. The contents were not deleted, but I lost focus and the paged was scrolled to the top.

Edit: They were.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Priority] High Used to indicate top priority items that need quick attention [Type] Bug An existing feature does not function as intended
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants