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

Enter key deletes single list item in a list, but leaves empty list #2753

Open
marinatells opened this issue Jan 15, 2019 · 1 comment
Open
Labels
plugin:list The plugin which probably causes the issue. status:confirmed An issue confirmed by the development team. type:bug A bug.

Comments

@marinatells
Copy link

Type of report

Bug

Provide detailed reproduction steps (if any)

  1. Enter https://nightly.ckeditor.com/basic/samples/
  2. Set data with developer tools
CKEDITOR.instances.editor.setData('<ul><li><br/></li></ul>')
  1. Place cursor manually inside empty list item
<ul><li><br/>^</li></ul>
  1. Press enter

Expected result

<p><br/></p>

Contents of li element are moved in a new node, and ul node is removed, because it's empty now and we don't need it anymore. Maybe it should have the same behaviour as when backspace is pressed?

Actual result

<p><br/></p><ul></ul>

UL tag stays in dom.

Other details

  • Browser: any
  • OS: any
  • CKEditor version: nightly
    Installed CKEditor plugins: enterkey, list
@jacekbogdanski jacekbogdanski self-assigned this Jan 16, 2019
@jacekbogdanski
Copy link
Member

I thought it may be a regression introduced in #2205 but it's not. There is no need to use a code to reproduce the issue, you can just add an empty list and press enter - a list still exists in the source code, but should be removed entirely.

@jacekbogdanski jacekbogdanski added type:bug A bug. status:confirmed An issue confirmed by the development team. plugin:list The plugin which probably causes the issue. labels Jan 16, 2019
@jacekbogdanski jacekbogdanski removed their assignment May 21, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
plugin:list The plugin which probably causes the issue. status:confirmed An issue confirmed by the development team. type:bug A bug.
Projects
None yet
Development

No branches or pull requests

2 participants