Opened 11 years ago

Closed 10 years ago

#11982 closed Bug (fixed)

Bullet added Above after enter key on indented bullet

Reported by: Nathan Frost Owned by: Artur Delura
Priority: Normal Milestone: CKEditor 4.4.8
Component: General Version: 4.2
Keywords: Cc:

Description (last modified by Piotrek Koszuliński)

Steps to reproduce (works on demo site):

  1. Create three items list.
  2. Place caret at the end of the last item.
  3. Press enter.
  4. Press tab to indent the new item.
  5. Press enter.
  • Expected Result: Last bullet should disappear, caret appears below list (or in the last item).
  • Actual Result: Editor adds new item after the 2nd one.

See Screencast: http://screencast.com/t/ggp0OoqUR (Note: screencast is misleading - read the description above).

Change History (8)

comment:1 Changed 11 years ago by Jakub Ś

Keywords: bullet cursor removed
Status: newconfirmed
Version: 4.3.34.2

NOTES:

  1. Screen cast is sort of unclear - please rely on description
  2. "Expected Result: Last bullet should disappear, cursor appears below list.""
    Expected result is that bullet on 1st level should appear.

Problem can be reproduced from CKEditor 4.2 in every browser.

comment:2 Changed 10 years ago by Gustav

Just stumbled upon this problem myself. Is there any update on this?

comment:3 Changed 10 years ago by Jakub Ś

Currently no one is working on this ticket but I will forward it, to my colleagues, as a reminder.

comment:4 Changed 10 years ago by Piotrek Koszuliński

Description: modified (diff)

comment:5 Changed 10 years ago by Artur Delura

Owner: set to Artur Delura
Status: confirmedassigned

comment:6 Changed 10 years ago by Artur Delura

Status: assignedreview

We decided that each enter command should outdent list one level. In ticket use case nesting was two so when enter is pressed we don't escape whole list, but just reduce indentation.

So having such situation

<ul>
    <li>foo
        <ul>
            <li>^</li>
        </ul>
    </li>
</ul>

and pressing enter we decrement intentation:

<ul>
    <li>foo</li>
    <li>^</li>
</ul>

That's the expected and implemented result in branch:t/11982b. Tests are also provided.

Last edited 10 years ago by Artur Delura (previous) (diff)

comment:7 Changed 10 years ago by Piotrek Koszuliński

Milestone: CKEditor 4.4.8

Assigning milestone, so we don't forget about this ticket.

comment:8 Changed 10 years ago by Piotr Jasiun

Resolution: fixed
Status: reviewclosed

I made some clean up in tests and closed with git:0db52ce.

Note: See TracTickets for help on using tickets.
© 2003 – 2022, CKSource sp. z o.o. sp.k. All rights reserved. | Terms of use | Privacy policy