Hidden paragraph BCD data still exists in content and in code

[peterbe]

In mdn/yari#1191 we made the text:

<div class="hidden">The compatibility table on this page is generated from structured data. If you'd like to contribute to the data, please check out <a href="https://github.com/mdn/browser-compat-data">https://github.com/mdn/browser-compat-data</a> and send us a pull request.</div>

be something that's part of the <BrowserCompatibilityTable> component. So every appearance of it in the content is redundant. This issue is about cleaning this up from the content.

[peterbe]

@nschonni Let's tackle this one once we've taken care of all the excess class="external" on all external links.

[peterbe]

It's clearly still there a lot:

MOZILLA/MDN/content  main ✔
▶ rg 'The compatibility table' | wc -l
    1247

[teoli2003]

I think this has to stay open until all are removed. I think it is my fault if it has been closed as I linked PR #5330 to it, even if it is a partial fix.

[peterbe]

Clearly some have been removed compared to by Feb 9 comment above:

▶ rg 'The compatibility table' | wc -l
    1222

[peterbe]

Bha! What's going on. I didn't mean for this to be closed. Sorry.

[teoli2003]

Clearly some have been removed compared to by Feb 9 comment above:

▶ rg 'The compatibility table' | wc -l
    1222

I noticed that it was near 1400 yesterday, meaning it was climbing. That's why I started removing them.

I guess people are cloning pages than modifying them.

[teoli2003]

We have removed the hidden paragraph BCD data from mdn/content.
rg 'The compatibility table' | wc -l 2

This 2 are not from such paragraph but from the contribute area explaining how to put compatibility tables in page.

@peterbe Is there still something to do here, or can we close this issue?

[peterbe]

There might be some manual searching where it has minor spelling differences that the python script can't pick up.
Otherwise, the next thing to do is to re-apply that Python script on mdn/translated-content :)

[teoli2003]

I ran the script with different needles, and manually fixed the remaining issues. :-), so I'm fairly confident for mdn/content.

I'm a bit afraid to look at the size of mdn/translated-content :-)

[peterbe]

Let's now redirect to mdn/translated-content#1008

[sideshowbarker]

I’m closing this here because per #2228 (comment) it seems to be resolved for the mdn/content repo — and because the follow-up issue for that mdn/translated-content repo has been raised at mdn/translated-content#1008.

But if I’m mistaken and this should in fact remain open, then somebody please do go ahead and re-open it.