Page MenuHomePhabricator

In an indented list, enter/return in a blank terminal li should move to a new li in the parent list, not a <p> in the current one
Closed, ResolvedPublic8 Estimated Story Points

Description

This may be related to T108357: Line breaks pasted as <br>s in Firefox.

Steps to reproduce:

  1. Type a new list:
* Food
** Apples
** Blueberries
** Cherries
  1. Then press Return (get a new "** list item") and Return again (get a new paragraph[1]). Type a new paragraph.
  1. Notice that the "new paragraph" is not properly aligned. The first character is aligned with the first character of the level-one list item, rather than on the left margin where it belongs.
  1. Save the page. Check the diff. The last item is ** List item new paragraph when it ought to be
** List item
New paragraph

[1] Why didn't pressing Return again un-indent to a "* list item"?

Event Timeline

Whatamidoing-WMF raised the priority of this task from to Needs Triage.
Whatamidoing-WMF updated the task description. (Show Details)
Jdforrester-WMF renamed this task from Paragraph after a list was added to the last list item to In an indented list, enter/return in a blank terminal li should move to a new li in the parent list, not a <p> in the current one.Aug 10 2015, 8:19 PM
Jdforrester-WMF triaged this task as Medium priority.
Jdforrester-WMF set Security to None.
Jdforrester-WMF edited a custom field.
Jdforrester-WMF removed a project: Parsoid.
Jdforrester-WMF subscribed.

Re-titling to be about the change in VE; I think Parsoid was doing the right thing here given the HTML VE sends.

Change 386418 had a related patch set uploaded (by DLynch; owner: DLynch):
[VisualEditor/VisualEditor@master] LinearEnterKeyDownHandler: Enter behavior in nested lists

https://gerrit.wikimedia.org/r/386418

Change 386418 merged by jenkins-bot:
[VisualEditor/VisualEditor@master] LinearEnterKeyDownHandler: Enter behavior in nested lists

https://gerrit.wikimedia.org/r/386418

Deskana assigned this task to DLynch.
Deskana changed the point value for this task from 1 to 8.

Change 386762 had a related patch set uploaded (by Jforrester; owner: Jforrester):
[mediawiki/extensions/VisualEditor@master] Update VE core submodule to master (8d1c0d026)

https://gerrit.wikimedia.org/r/386762

Change 386762 merged by jenkins-bot:
[mediawiki/extensions/VisualEditor@master] Update VE core submodule to master (8d1c0d026)

https://gerrit.wikimedia.org/r/386762