Last modified: 2014-07-23 21:05:59 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 T70466, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 68466 - Add parser test for multiline <pre> or <syntaxhighlight> elements inside a dd element created by ":"
Add parser test for multiline <pre> or <syntaxhighlight> elements inside a dd...
Status: NEW
Product: Parsoid
Classification: Unclassified
General (Other open bugs)
unspecified
All All
: Normal normal
: ---
Assigned To: Parsoid Team
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2014-07-23 19:24 UTC by Helder
Modified: 2014-07-23 21:05 UTC (History)
2 users (show)

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


Attachments

Description Helder 2014-07-23 19:24:36 UTC
Please, add some parser tests to ensure that Parsoid interprets the wiki markup
;A
:B<syntaxhighlight lang="CSS">
.test {
    color: red;
}
</syntaxhighlight>
as having 'an element <dd> which contains "B" and the highlighted code', instead of what the current parser does:
https://www.mediawiki.org/w/index.php?title=Project:Sandbox&oldid=1075316&action=edit&preview=yes

Here is a comparison of the wiki markup and HTML markup with the current parser:
https://www.mediawiki.org/w/index.php?title=Project:Sandbox&oldid=1075311

And here is the same kind of problem with <pre> tags instead of <syntaxhighlight>:
https://www.mediawiki.org/w/index.php?title=Project:Sandbox&oldid=1075317&action=edit&preview=yes
The current parser converts the code
;A
:B<pre>first line
second line</pre>
to
<dl>
<dt>A</dt>
<dd>B
<pre>first line
</pre></dd>
</dl>
<p>second line</p>
but it should be converted to
<dl>
<dt>A</dt>
<dd>B
<pre>first line
second line
</pre></dd>
</dl>
as Parsoid does:
http://parsoid-lb.eqiad.wikimedia.org/mediawikiwiki/Project:Sandbox?oldid=1075317
Comment 1 ssastry 2014-07-23 20:39:03 UTC
The second test can be added right away -- there are a number of dl/dd list tests in a section. Your test can be added there. Would you like to submit a patch to parserTests.txt in the parsoid repo? If not, one of us will pick it up in the coming days.

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


Navigation
Links