Last modified: 2013-05-01 23:18:48 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 T47023, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 45023 - Parsoid: whitespace preservation when serializing edited content completely broken
Parsoid: whitespace preservation when serializing edited content completely b...
Status: RESOLVED FIXED
Product: Parsoid
Classification: Unclassified
serializer (Other open bugs)
unspecified
All All
: Unprioritized normal
: ---
Assigned To: Gabriel Wicke
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-02-14 22:03 UTC by Roan Kattouw
Modified: 2013-05-01 23:18 UTC (History)
3 users (show)

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


Attachments

Description Roan Kattouw 2013-02-14 22:03:24 UTC
/_rt/ doesn't catch this, but downloading and resubmitting the HTML does, as does using the editor.

Create a page with the following wikitext:

Foo bar

Bar baz 

== Baz quux ==
Quux whee


To reproduce with the editor:
1. Open the page in the editor
2. Make "bar" bold
3. Click "Review and save"
4. The resulting diff will remove the newline between "Bar baz" and "Quux whee", and add a newline between "Baz quux" and "Quux whee"

To reproduce without the editor:
1. Download /localhost/Pagename from Parsoid
2. Submit the resulting HTML back to /_html/
3. The output will be:

Foo bar

Bar baz
== Baz quux ==

Quux whee
Comment 1 Gabriel Wicke 2013-02-15 00:52:44 UTC
This is a known issue with the finer-grained selective serializer and separator handling in the WTS, and the reason why we rolled back the last deployment. The serializer is currently being refactored to use DOM-based handlers instead of the current token-based ones, which should fix separator issues like this one.
Comment 2 Gabriel Wicke 2013-05-01 23:18:48 UTC
Fixed with current Parsoid and VE master.

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links