What's with the <nowiki></nowiki>s? I'm less concerned about the extra wikitable class going away.

@MZMcBride the <nowiki> are required if you want the contents of the table cell to be -. Note that there are existing cells with <nowiki> of the same sort.

Yeah, I saw that there were other similar cells. I misread <nowiki>-</nowiki> as <nowiki></nowiki>. My bad.

From quick testing, it seems !- creates a <th> cell containing a "-" and | - creates a <td> cell containing a "-". The latter seems more obvious to me and less clunky than wrapping nowiki tags around "-", but shrug.

I agree that serializing as "| -" might be more 'natural'. It's possible we could hack that into Parsoid as a special case. That's a separate bug/feature request though.

Change 191037 had a related patch set uploaded (by Esanders):
Test for class attribute duplication bug

https://gerrit.wikimedia.org/r/191037

Patch-For-Review

Change 191028 had a related patch set uploaded (by Catrope):
Ignore duplicates when comparing class lists

https://gerrit.wikimedia.org/r/191028

Patch-For-Review

Change 191028 merged by jenkins-bot:
Ignore duplicates when comparing class lists

https://gerrit.wikimedia.org/r/191028

I can't understand why a duplicate class name is acceptable. Of course you must ignore these duplicates but the editor should also simply discard them

Change 191037 merged by jenkins-bot:
Test for class attribute duplication bug

https://gerrit.wikimedia.org/r/191037