VisualEditor: [Regression] Categories and DEFAULTSORT meta items getting modified to strip default whitespace?
OpenPublic

Description

Sometimes editing a page in VisualEditor is rearranging categories so that they always end up on a single line, which is inconvenient for people looking at the wikitext source later.

See https://fr.wikipedia.org/w/index.php?title=Jean-Baptiste_Vallière&diff=prev&oldid=104017086


Version: unspecified
Severity: normal

bzimport added a project: VisualEditor-DataModel.Via ConduitNov 22 2014, 3:15 AM
bzimport set Reference to bz65647.
Whatamidoing-WMF created this task.Via LegacyMay 22 2014, 6:16 PM
Jdforrester-WMF added a comment.Via ConduitMay 22 2014, 6:17 PM

Parsoid bug? We hopefully aren't dirtying these, but even if we are, Parsoid's default whitespace should insert a \n before each of these anyway…

GWicke added a comment.Via ConduitMay 22 2014, 6:22 PM

We only use the default newline spacing if the categories are actually new content. So if VE is just removing the newlines, but leaves the categories, a result as in the link is to be expected.

Round-tripping works fine with selser:

http://parsoid-lb.eqiad.wikimedia.org/_rtselser/frwiki/Jean-Baptiste_Valli%C3%A8re?oldid=95815820

Even without selser the newlines are preserved:

http://parsoid-lb.eqiad.wikimedia.org/_rt/frwiki/Jean-Baptiste_Valli%C3%A8re?oldid=95815820

This makes it very likely that this is in fact VE removing newlines.

NicoV added a comment.Via ConduitMay 24 2014, 8:12 AM

Other example that also displays other problems (extra characters at the beginning, and strange characters in the text ☃☃)
https://fr.wikipedia.org/w/index.php?title=Absolom_2022&diff=104059827&oldid=96660949

Jdforrester-WMF moved this task to Backlog on the VisualEditor workboard.Via WebNov 24 2014, 1:27 AM
Jdforrester-WMF lowered the priority of this task from "High" to "Normal".Via WebJan 15 2015, 12:29 AM
Jdforrester-WMF set Security to None.

Add Comment

Column Prototype
This is a very early prototype of a persistent column. It is not expected to work yet, and leaving it open will activate other new features which will break things. Press "\" (backslash) on your keyboard to close it now.