Wikimedia Bugzilla is closed!

While Parsoid's rendering is obviously not correct (this isn't a VE-only issue, the raw rendering shows the same problems: http://parsoid.wmflabs.org/nlwiki/Justine_Henin?oldid=41676266), the template could easily be simplified.

There are many templates that generate partial table cell markup on various wikis that work, so it's just some quirk of this one that Parsoid is unable to understand.

Here's a version of the template with simpler markup that will probably be parsed correctly (but I haven't tested). It has less # and less {{!}} :).

I didn't want to modify it on the wiki in case it actually broke something (I tried it with some pages and it was okay), feel free to use this code.

(I don't work on Parsoid, I just found this interesting.)

align="center" {{#switch: {{{1|}}}
| -        = style="background: #ffffff;"
| G        = style="background: #def8fe;"
| 1R       = style="background: #def8fe;"
| 2R       = style="background: #bef1fe;"
| 3R       = style="background: #8cecfd;"
| 4R       = style="background: #68dffd;"
| KF       = style="background: #ffebcd;"
| HF       = style="background: #dddd00;"
| F        = style="background: #e4c8e4;"
| W        = style="background: #00ff00;"
| #default = style="background: #f9f9f9;"
}} | {{#if: {{{2|}}}
| {{#ifeq: {{{1}}} | W | '''[[{{{2}}}|{{{1}}}]]''' | [[{{{2}}}|{{{1}}}]] }}
| {{#ifeq: {{{1}}} | W | '''{{{1}}}''' | {{{1}}} }}
}}

(In reply to Bartosz Dziewoński from comment #1)

> http://parsoid.wmflabs.org/nlwiki/Justine_Henin?oldid=41676266), the
> template could easily be simplified.

Thanks for looking into it. You suggest to consider simplifying the template. Do you have a link to a list of  partial table cell markup templates and/or a link with suggestions how to simplify the template?

After bwc: you've already coded a possible simplification. I will test that one. Thanks a lot!

(In reply to Ad Huikeshoven from comment #3)
> Do you have a link to a list of  partial table cell markup
> templates and/or a link with suggestions how to simplify the template?

Here's a list of ones used on en.wikipedia (probably not a full list): https://en.wikipedia.org/wiki/Template:Yes#Templates – these are in general simpler than the one we're talking about here, but they seem to all render correctly in Parsoid (http://parsoid.wmflabs.org/enwiki/Template%3AYes?oldid=600725279).

https://nl.wikipedia.org/wiki/Gebruiker:Ad_Huikeshoven/jh?veaction=edit shows that the modified template https://nl.wikipedia.org/wiki/Sjabloon:Tabelcel_tennis/vereenvoudigd resolves the issue for at least one page.

(Let's leave this open for a while and let Parsoid team look, maybe the underlying issue in Parsoid parser is easily fixable :) )

(VE has some fun issues with these templates, filed as bug 67856 and bug 67857.)

I think the problem is with the # entity that is showing up in the table-css. Parsoid wraps entities in additional HTML markup which gets in the way of Parsoid's strategy to handle these templates.

[subbu@earth tests] cat /tmp/tbl
{|
| {{Tabelcel tennis|X}}
| {{Tabelcel tennis/vereenvoudigd|X}}
|}

In the output below, the background css gets broken up by the entity that is encountered there. Entities are not a problem in html attributes in general, but only where the table-cell attribute is first parsed as text (in templates like this where the table-cell and its css comes from 2 different places) and later patched up. I think fixing up the template is a better solution rather than add additional logic in Parsoid to undo the entity wrapping in these cases.

[subbu@earth tests] node parse --prefix nlwiki --dump tplsrc --trace peg < /tmp/tbl

....
=================================
Sjabloon:Tabelcel_tennis/vereenvoudigd
---------------------------------
align="center" style="background: #f9f9f9;" | X
---------------------------------
1-[peg]        | ---->   ["align=\"center\" style=\"background: #f9f9f9;\" | X"]
1-[peg]        | ---->   [{"type":"EOFTk"}]
=================================
Sjabloon:Tabelcel_tennis
---------------------------------
align="center" style="background:&#35;F9F9F9;"|X
---------------------------------
2-[peg]        | ---->   ["align=\"center\" style=\"background:",{"type":"TagTk","name":"span","attribs":[{"k":"typeof","v":"mw:Entity"}],"dataAttribs":{"src":"&#35;","srcContent":"#","tsr":[33,33]}},"#",{"type":"EndTagTk","name":"span","attribs":[],"dataAttribs":{"tsr":[38,38]}},"F9F9F9;\"|X"]
2-[peg]        | ---->   [{"type":"EOFTk"}]
....

(Should this be closed as WONTFIX or WORKSFORME or something then?)

(In reply to Bartosz Dziewoński from comment #10)
> (Should this be closed as WONTFIX or WORKSFORME or something then?)

Hmm .. I dont know. Marking this WONTFIX can be seen as an antagonistic stance and hence my hesitation to mark it as such right now. We are not closing the door shut on this in case a cogent argument can be made for why this is a good investment of developer time instead of doing the 1-off update of the template and be done with this. It is better for nlwiki folks or other active editors/admins to close this as WORKSFORME after updating the template on nlwiki.

Here what looks like to be a pretty bad looking example on idwiki:
http://parsoid-lb.eqiad.wikimedia.org/idwiki/Nickelodeon_Kids%27_Choice_Awards?oldid=8237629

I'm not sure to really understand/agree with the "lowest" priority for this bug as the impact is not low at all.

Kelson: your example is a bit different from the nlwiki scenario. Looks like we have an unhandled edge case. In the example below, the first row is handled, but the second isn't. Should be easy to fix.

{|
|-
!nowrap|Favorite Movie
| {{yes}}
|<!-- 1989 --> {{yes}}
|}

(In reply to ssastry from comment #13)
> Kelson: your example is a bit different from the nlwiki scenario. Looks like
> we have an unhandled edge case. In the example below, the first row is
> handled, but the second isn't. Should be easy to fix.
> 
> {|
> |-
> !nowrap|Favorite Movie
> | {{yes}}
> |<!-- 1989 --> {{yes}}
> |}

Fixed as bug 72487

@ssastry
Yes 100% fixed. Thank you!