Page MenuHomePhabricator

VisualEditor: <pre> tags have leading whitespace removed
Closed, ResolvedPublic

Description

Around the time when VE started to support editing of pres we started to get dirty diffs like this:

http://www.mediawiki.org/w/index.php?title=Parsoid&diff=779292&oldid=777387

This might either be a selser issue, or VE dirtying the DOM.


Version: unspecified
Severity: normal

Details

Reference
bz54083

Event Timeline

bzimport raised the priority of this task from to Medium.Nov 22 2014, 2:13 AM
bzimport set Reference to bz54083.

There's a check tag on the edit, so VE dirtied something somewhere. That may or may not explain the whole bug, but we should fix it.

Change 84332 had a related patch set uploaded by Esanders:
Fix check for preformatted when stripping whitespace

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

Change 84332 merged by jenkins-bot:
Fix check for preformatted when stripping whitespace

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

Change 84896 had a related patch set uploaded by Catrope:
Fix check for preformatted when stripping whitespace

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

Change 84896 merged by Catrope:
Fix check for preformatted when stripping whitespace

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

  • Bug 53775 has been marked as a duplicate of this bug. ***