Last modified: 2013-05-16 16:11:26 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T50287, the corresponding Phabricator task for complete and up-to-date bug report information.
First Last Prev Next    This bug is not in your last search results.
Bug 48287 - VisualEditor: Aborted empty list creates a pawn
VisualEditor: Aborted empty list creates a pawn
Status: RESOLVED FIXED
Product: VisualEditor
Classification: Unclassified
ContentEditable (Other open bugs)
unspecified
All All
: Highest normal
: VE-deploy-2013-05-13
Assigned To: Roan Kattouw
:
Depends on:
Blocks:
  Show dependency tree / graph
 
Reported: 2013-05-09 08:59 UTC by Nicolas Raoul
Modified: 2013-05-16 16:11 UTC (History)
5 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---

Attachments
Add an attachment (proposed patch, testcase, etc.)

Description Nicolas Raoul 2013-05-09 08:59:55 UTC 
Go to http://en.wikipedia.org/wiki/Content_Management_Interoperability_Services
Put the cursor somewhere in the text.
Type "Enter" 2 times.
Type "Up".
Click Bullet List
Type Enter
Type Up
Type Delete

Wild pawn appears.
Comment 1 Nicolas Raoul 2013-05-09 09:02:15 UTC 
Same when entering text instead of Delete
Comment 2 James Forrester 2013-05-09 19:13:53 UTC 
I can reproduce in Firefox and Chrome if you forwards-delete or insert characters. It appears that this is because when you clear out the <li> (return at the end of the list) it doesn't check that the <ul> is empty, so you get a node inside the <ul>.

Note that in casual testing this 'worked', because backspace deletes the pawn and <ul> together. Bother.
Comment 3 Gerrit Notification Bot 2013-05-10 01:14:15 UTC 
Related URL: https://gerrit.wikimedia.org/r/63102 (Gerrit Change If22d9b904b8861e24d56944d791545635b2e4254)
Comment 4 Roan Kattouw 2013-05-10 17:17:53 UTC 
(In reply to comment #3)
> Related URL: https://gerrit.wikimedia.org/r/63102 (Gerrit Change
> If22d9b904b8861e24d56944d791545635b2e4254)

Merged.
Comment 5 James Forrester 2013-05-10 17:19:09 UTC 
Merged into master; will be deployed from Monday.
Note You need to log in before you can comment on or make changes to this bug.
First Last Prev Next    This bug is not in your last search results.

Navigation
Links