Can we bikeshed on a proper name? "stx: html" isn't really intuitive for an externally-visible API.

data-mw='{"repr":"html"}' or data-mw-repr="html"? repr being short for representation. But that's not terribly better than stx. Better ideas?

The values are html and wt? Is it an observation or a command to Parsoid? If the latter, should it be serialiseAs or something?

<b serializeAs="html">
<b serializeAs="wt">

Of course one or the other of those (the latter?) would presumably be the default and wouldn't need to be spelled out.

In T100856#2019808, @Jdforrester-WMF wrote:

The values are html and wt? Is it an observation or a command to Parsoid? If the latter, should it be serialiseAs or something?

Like all data-mw info, it is read/write. It is best to think of it as a property declaration.

Alternative to 'stx': 'html" would be 'html': true

Yeah, though the serialisation format might (?!) differ between tags, e.g. <video>? Also <b serializeAs="md"> for future-proofing. :-)

I am going to veto the 'serializeAs' attribute. Let us continue to use declarations.

@ssastry oh, agreed. I just spelled it as an attribute rather than embedded within data-mw because I was lazy about typing.

<b data-mw='{"serializeAs": "html" }'>

But I'm not sure I agree with @Jdforrester-WMF that this should have the imperative tense and be a "command to Parsoid". This is also output from the wt2html pass. It ought to describe the content's preferred format, not command parsoid.

But I also agree with @Jdforrester that we might want to leave room for future fine distinctions in wikitext presentation. Maybe format is a good name? format=html and format=wt, with future expandability for tags which can be serialized in multiple ways. Maybe even format=wt-singleline or something like that to handle some whitespace issues?

This is also output from the wt2html pass.

Yes, but in the context of "on round-trip you must re-serialise as this lest we break things", right?

In T100856#2019851, @Jdforrester-WMF wrote:

This is also output from the wt2html pass.

Yes, but in the context of "on round-trip you must re-serialise as this lest we break things", right?

I'm not totally sure. You might have a third party data source of Parsoid-format DOM, where you're just indicating "this was originally an html tag" or "this will be more readable if you format it in the following way when rendered as wikitext". And serializeAs implies there's only one serialization (to wikitext) -- it might be more forward-thinking to scope this as a hint about *wikitext* serialization specifically.

Note that this patch, whenever it is written and deployed, could result in a complete re-rendering of a lot of the stored content in RESTBase .. since most pages have some html tag or the other. The thing to check is if we set these attributes on templated content. If not, then fewer pages will be affected, but nevertheless a large number of pages are likely to get re-rendered.

@ssastry Can you clarify if any action is needed from the Language team, or if this will affect some of our products?

Change #1115957 had a related patch set uploaded (by Arlolra; author: Arlolra):

[mediawiki/services/parsoid@master] Keep stx info for all headings

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

Change #1115957 merged by jenkins-bot:

[mediawiki/services/parsoid@master] Keep stx info for all headings

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

I'm voting to move this into data-mw, or even typeof, since "is this a 'real' heading or not" has started to become relevant for a number of "read view" things, like "should we add section edit links beside this".

We don't need to break all HTML: we could add this as an "extra" property first, and look both at the extra property and at the data-parsoid when serializing. Since data-parsoid is considered "private", if/when we eventually remove it from the data-parsoid that won't necessarily be a breaking change.

Change #1116870 had a related patch set uploaded (by Jgiannelos; author: Jgiannelos):

[mediawiki/vendor@master] Bump wikimedia/parsoid to 0.21.0-a14

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

Change #1116870 merged by jenkins-bot:

[mediawiki/vendor@master] Bump wikimedia/parsoid to 0.21.0-a14

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

Change #1302971 had a related patch set uploaded (by C. Scott Ananian; author: C. Scott Ananian):

[mediawiki/services/parsoid@master] Add `data-mw-wikitext` to wikitext-syntax headings

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

Change #1302972 had a related patch set uploaded (by C. Scott Ananian; author: C. Scott Ananian):

[mediawiki/core@master] HandleParsoidSectionLinks: look for data-mw-wikitext marker attribute

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

Change #1302972 merged by jenkins-bot:

[mediawiki/core@master] HandleParsoidSectionLinks: look for data-mw-wikitext marker attribute

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