Skip to content

Trigger renumbering when deleting (via cut) the first item of a list#2533

Merged
JohnMcLear merged 1 commit intoether:developfrom
cjb:issue-2514-renumber-on-cut
Feb 27, 2015
Merged

Trigger renumbering when deleting (via cut) the first item of a list#2533
JohnMcLear merged 1 commit intoether:developfrom
cjb:issue-2514-renumber-on-cut

Conversation

@cjb
Copy link
Copy Markdown
Contributor

@cjb cjb commented Feb 27, 2015

Fixes #2514.

@JohnMcLear
Copy link
Copy Markdown
Member

Wow this is crazy clean.. Didn't expect this.. Does it work on pad "viewers" as well as the authors screen?

@cjb
Copy link
Copy Markdown
Contributor Author

cjb commented Feb 27, 2015

@JohnMcLear Yep, seems to work fine for both the author and viewers immediately.

@JohnMcLear
Copy link
Copy Markdown
Member

Wow excellent! I will give it a test..

JohnMcLear added a commit that referenced this pull request Feb 27, 2015
Trigger renumbering when deleting (via cut) the first item of a list
@JohnMcLear JohnMcLear merged commit cc0eaba into ether:develop Feb 27, 2015
@JohnMcLear
Copy link
Copy Markdown
Member

Hrm, the caret appears to jump now to the last line on delete, are you seeing that?

@cjb
Copy link
Copy Markdown
Contributor Author

cjb commented Feb 27, 2015

No, don't think I can reproduce that -- is it easy for you to share a screencast?

It's a little weird to call doDeleteKey() when no key was pressed down, so it might be related to that, but I wasn't able to find any misbehavior with it here.

@lid2000 lid2000 mentioned this pull request Mar 12, 2015
@webzwo0i webzwo0i mentioned this pull request Dec 15, 2020
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.

Deleting the first item of a numbered list causes all items to be number one

2 participants