Change Details

At least cebwiki and svwiki seem to be using this pattern of embedding a table inside another table in fosterable position and relying on Tidy doing some weird fixup that inverts the order of the tables. Ex: https://sv.wikipedia.org/wiki/Kugelstein and https://ceb.wikipedia.org/wiki/Wujieyue_Shan. svwiki pages have this wikitext: ``` {| border="1" {{klimatöversikt ... }} |} ``` and cebwiki pages have this: ``` {| border="1" {{climate chart ... }} |} ``` In both cases, we effectively have HTML of the form: ``` <table border="1"> <---- GENERATED BY THE {| WIKITEXT <table class="infobox" style="width: 19.5em; float: left; clear: left; margin-left:0em;margin-right:1em; text-align: center; border: solid 1px silver" cellspacing="0" cellpadding="0"> <---- GENERATED BY THE TEMPLATE ... </table> </table> ``` This is broken HTML and a HTML5 parser (RemexHTML, HTML5Depurate, Balancer, Parsoid) will generate output of the form: ``` <table border="1"></table> <--- EMPTY TABLE <table class="infobox" style="width: 19.5em; float: left; clear: left; margin-left:0em;margin-right:1em; text-align: center; border: solid 1px silver" cellspacing="0" cellpadding="0"> .... table content here ... </table> ``` However, Tidy removes the infobox-class based table skeleton and retains the table border="1" in place yielding: ``` <table border="1"> ... table content here ... </table> ``` Given this Tidy output (where the templates outermost table shell is removed), a good way to get similar output in Tidy, RemexHTML, and Parsoid would be to edit the templates to remove the opening <table> and closing </table> from the [[https://ceb.wikipedia.org/w/index.php?title=Template:climate%20chart&action=edit|cebwiki:climate chart]] and the [[https://sv.wikipedia.org/w/index.php?title=Mall:Klimat%C3%B6versikt&action=edit|svwiki:klimatöversikt]] and other such templates on other wikis where this pattern is used. This difference has been found in the Tidy replacement project tests (row 3 of [[https://www.mediawiki.org/wiki/Parsing/Replacing_Tidy#2._Wikitext_markup_errors|Replacing Tidy Test Results]]) and while comparing PHP parser output and Parsoid output. A good way to do this would be to: 1. Edit the relevant template and remove the opening <table> and closing </table> 2. Edit the template on one of these wikis 3. Purge an affected page via ?action=purge 4. Verify that the new output is still similar. 5. Verify that Parsoid output now matches the Tidy output (by going to https://<wiki>.wikimedia.org/api/rest_v1/page/html/<title> and force-reloading the page).

At least cebwiki and svwiki seem to be using this pattern of embedding a table inside another table in fosterable position and relying on Tidy doing some weird fixup that inverts the order of the tables. Ex: https://sv.wikipedia.org/wiki/Kugelstein and https://ceb.wikipedia.org/wiki/Wujieyue_Shan. svwiki pages have this wikitext: ``` {| border="1" {{klimatöversikt ... }} |} ``` and cebwiki pages have this: ``` {| border="1" {{climate chart ... }} |} ``` In both cases, we effectively have HTML of the form: ``` <table border="1"> <---- GENERATED BY THE {| WIKITEXT <table class="infobox" style="width: 19.5em; float: left; clear: left; margin-left:0em;margin-right:1em; text-align: center; border: solid 1px silver" cellspacing="0" cellpadding="0"> <---- GENERATED BY THE TEMPLATE ... </table> </table> ``` This is broken HTML and a HTML5 parser (RemexHTML, HTML5Depurate, Balancer, Parsoid) will generate output of the form: ``` <table border="1"></table> <--- EMPTY TABLE <table class="infobox" style="width: 19.5em; float: left; clear: left; margin-left:0em;margin-right:1em; text-align: center; border: solid 1px silver" cellspacing="0" cellpadding="0"> .... table content here ... </table> ``` However, Tidy removes the infobox-class based table skeleton and retains the table border="1" in place yielding: ``` <table border="1"> ... table content here ... </table> ``` Given this Tidy output (where the templates outermost table shell is removed), a good way to get similar output in Tidy, RemexHTML, and Parsoid would be to edit the templates to remove the opening <table> and closing </table> from the [[https://ceb.wikipedia.org/w/index.php?title=Template:climate%20chart&action=edit|cebwiki:climate chart]] and the [[https://sv.wikipedia.org/w/index.php?title=Mall:Klimat%C3%B6versikt&action=edit|svwiki:klimatöversikt]] and other such templates on other wikis where this pattern is used. This difference has been found in the Tidy replacement project tests (row 3 of [[https://www.mediawiki.org/wiki/Parsing/Replacing_Tidy#2._Wikitext_markup_errors|Replacing Tidy Test Results]]) and while comparing PHP parser output and Parsoid output. A good way to do this would be to: 1. Edit the relevant template on one of these wikis by removing the opening <table> and closing </table> 2. Purge an affected page via ?action=purge 3. Verify that the new output is still similar. 4. Verify that Parsoid output now matches the Tidy output (by going to https://<wiki>.wikimedia.org/api/rest_v1/page/html/<title> and force-reloading the page).

At least cebwiki and svwiki seem to be using this pattern of embedding a table inside another table in fosterable position and relying on Tidy doing some weird fixup that inverts the order of the tables. Ex: https://sv.wikipedia.org/wiki/Kugelstein and https://ceb.wikipedia.org/wiki/Wujieyue_Shan. svwiki pages have this wikitext: ``` {| border="1" {{klimatöversikt ... }} |} ``` and cebwiki pages have this: ``` {| border="1" {{climate chart ... }} |} ``` In both cases, we effectively have HTML of the form: ``` <table border="1"> <---- GENERATED BY THE {| WIKITEXT <table class="infobox" style="width: 19.5em; float: left; clear: left; margin-left:0em;margin-right:1em; text-align: center; border: solid 1px silver" cellspacing="0" cellpadding="0"> <---- GENERATED BY THE TEMPLATE ... </table> </table> ``` This is broken HTML and a HTML5 parser (RemexHTML, HTML5Depurate, Balancer, Parsoid) will generate output of the form: ``` <table border="1"></table> <--- EMPTY TABLE <table class="infobox" style="width: 19.5em; float: left; clear: left; margin-left:0em;margin-right:1em; text-align: center; border: solid 1px silver" cellspacing="0" cellpadding="0"> .... table content here ... </table> ``` However, Tidy removes the infobox-class based table skeleton and retains the table border="1" in place yielding: ``` <table border="1"> ... table content here ... </table> ``` Given this Tidy output (where the templates outermost table shell is removed), a good way to get similar output in Tidy, RemexHTML, and Parsoid would be to edit the templates to remove the opening <table> and closing </table> from the [[https://ceb.wikipedia.org/w/index.php?title=Template:climate%20chart&action=edit|cebwiki:climate chart]] and the [[https://sv.wikipedia.org/w/index.php?title=Mall:Klimat%C3%B6versikt&action=edit|svwiki:klimatöversikt]] and other such templates on other wikis where this pattern is used. This difference has been found in the Tidy replacement project tests (row 3 of [[https://www.mediawiki.org/wiki/Parsing/Replacing_Tidy#2._Wikitext_markup_errors|Replacing Tidy Test Results]]) and while comparing PHP parser output and Parsoid output. A good way to do this would be to: 1. Edit the relevant template andon one of these wikis by removeing the opening <table> and closing </table> 2. Edit the template on one of these wikisPurge an affected page via ?action=purge 3. Purge an affected page via ?action=purgeVerify that the new output is still similar. 4. Verify that the new output is still similar. 5. Verify that Parsoid output now matches the Tidy output (by going to https://<wiki>.wikimedia.org/api/rest_v1/page/html/<title> and force-reloading the page).