Page MenuHomePhabricator

VisualEditor: Trying to un-bullet a bulleted line containing only floated content makes the browser completely unresponsive
Closed, ResolvedPublic

Description

Steps to reproduce:

1.Open any article on VE
2.Add a bullet list
3.Try to add an image as a bulleted item of that list ( the image goes to the right side but keep it like that)
4.Save the page
5.Reopen the editor
6.Now go to position where empty bullet list is.
7.Now try to remove it by clicking ob bullet list icon from the toolbar

Observed Result:
The Editor become completely unresponsive


Version: unspecified
Severity: normal

Details

Reference
bz58060

Event Timeline

bzimport raised the priority of this task from to High.Nov 22 2014, 2:23 AM
bzimport set Reference to bz58060.

Infact it makes the browser page unresponsive.I could not close the browser window even until it showed an error to kill page in Chrome and stop script in FF.

See the screenshot attached.

Test Environment: production
Browser: Chrome Version 26.0.1410.65 & FF 25
OS: MAC OS X 10. 8. 5

Example Page:https://en.wikipedia.org/wiki/User:RYasmeen_%28WMF%29/unresponsive?veaction=edit

In this page, Go to the last bullet and click on Bullet icon from toolbar to reproduce.

Infact it makes the browser page unresponsive.I could not close the browser window even until it showed an error to kill page in Chrome and stop script in FF.

See the screenshot attached.

Test Environment: production
Browser: Chrome Version 26.0.1410.65 & FF 25
OS: MAC OS X 10. 8. 5

Example Page:https://en.wikipedia.org/wiki/User:RYasmeen_%28WMF%29/unresponsive?veaction=edit

In this page, Go to the last bullet and click on Bullet icon from toolbar to reproduce.

Created attachment 13998
Screenshot

Attached:

Screen_Shot_2013-12-05_at_2.52.55_PM.png (343×1 px, 109 KB)

Also occurs (same STR) with the wikitext

  • {{infobox}}

on testwiki.

Change 115385 had a related patch set uploaded by Esanders:
Check indentation was successful before trying again

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

Above patch prevents infinite loop. Need to investigate why indentation is failing.

Change 115402 had a related patch set uploaded by Esanders:
Add special case to de-indentation for slugs in listItems

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

Change 115385 merged by jenkins-bot:
Check indentation was successful before trying again

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

Change 115402 merged by jenkins-bot:
Add special case to de-indentation for slugs in listItems

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