browser-compat-data icon indicating copy to clipboard operation
browser-compat-data copied to clipboard

Mark the item* microdata attributes as not supported by browsers

Open foolip opened this issue 1 year ago • 5 comments

The API reflecting these attributes was removed already: https://github.com/mdn/browser-compat-data/pull/16226

The attributes have never had any effect beyond the API for accessing them, so it's fair to say these attributes are not supported.

A search for "itemscope" in the source of Chromium, Gecko and WebKit finds only old tests and similar, no code handling the attribute.

foolip avatar Jan 23 '24 14:01 foolip

@Elchi3 they are documented on MDN and I think that makes sense since they're used by https://schema.org/docs/gs.html and probably other things. Should content be updated to just not have a BCD key then?

foolip avatar Jan 24 '24 14:01 foolip

Should content be updated to just not have a BCD key then?

hm, I guess I would remove the compat tables on MDN and leave a note there saying that browsers do not necessarily need to support this. I mean so that absent a compat table readers are not wondering about the compat story.

Elchi3 avatar Jan 24 '24 15:01 Elchi3

I mean an all-false table also leaves the wrong impression that this is unusable, right?

Elchi3 avatar Jan 24 '24 15:01 Elchi3

Just removing the compat tables sounds good to me. Do you want me to do that before we move forward with this PR then?

foolip avatar Feb 03 '24 23:02 foolip

That would be great, yes! Thank you!

Elchi3 avatar Feb 05 '24 17:02 Elchi3

I've sent https://github.com/mdn/content/pull/32926 to drop the compat tables. Should I just remove these entries from BCD then?

foolip avatar Apr 02 '24 15:04 foolip

Yes, please remove them in this PR.

And as Will says in https://github.com/mdn/content/pull/32926#pullrequestreview-1975208128 use mdn/content's front-matter yaml to provide the spec url for these features in content directly then.

Elchi3 avatar Apr 04 '24 10:04 Elchi3

https://github.com/mdn/content/pull/32926 is merged now, so this should be good to go too.

foolip avatar Apr 05 '24 18:04 foolip