Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update review of monitored specs #1477

Merged
merged 3 commits into from
Sep 3, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 8 additions & 0 deletions specs.json
Original file line number Diff line number Diff line change
Expand Up @@ -676,6 +676,7 @@
},
"https://wicg.github.io/datacue/",
"https://wicg.github.io/deprecation-reporting/",
"https://wicg.github.io/digital-credentials/",
"https://wicg.github.io/digital-goods/",
"https://wicg.github.io/digital-identities/",
"https://wicg.github.io/direct-sockets/",
Expand Down Expand Up @@ -725,6 +726,7 @@
"shortname": "speculation-rules"
},
"https://wicg.github.io/netinfo/",
"https://wicg.github.io/observable/",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it's not shipping anywhere yet though? (although somehow WPT shows support in Cr 130?)

not opposed to adding it, but this changes a bit our current expectations…

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

But then "shipping in a browser" is not in our list of criteria. For this and a few other entries under incubation, I don't really know how to evaluate readiness to enter the list. In this case, "arguments" in favor of adding it to the list:

Arguments against adding it to the list:

  • No involvement from other browser engines for now
  • Tests in WPT are flagged as "tentative"
  • No documentation on MDN

Essentially, I've been flipping a coin. I'm happy to flip it again and keep it in the monitor list.

I would find it useful to review and clarify the spec selection criteria in light of accumulated experience. I'll track that in a separate issue.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I find the pro arguments convincing, so let's add it and continue the criteria discussion separately

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Criteria discussion in #1481

"https://wicg.github.io/overscroll-scrollend-events/",
"https://wicg.github.io/page-lifecycle/",
"https://wicg.github.io/performance-measure-memory/",
Expand Down Expand Up @@ -1276,6 +1278,12 @@
"https://www.w3.org/TR/html-aam-1.0/",
"https://www.w3.org/TR/html-aria/",
"https://www.w3.org/TR/html-media-capture/",
{
"url": "https://www.w3.org/TR/html-ruby-extensions/",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hmm... This is in practice a delta spec of the HTML spec (although not intended to serve as a next iteration of it); should this be represented somehow?

In particular, it adds 2 HTML elements that I suspect we want to detect.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is in practice a delta spec of the HTML spec (although not intended to serve as a next iteration of it); should this be represented somehow?

I was thinking we would handle that as we handle TC39 proposals: at some point in the future, we flag the entry as discontinued and obsoleted by HTML. It's not easy to flag the spec as being a "delta" of "HTML" (code would get confused by a seriesPrevious property in specs.json in particular)

In particular, it adds 2 HTML elements that I suspect we want to detect.

They should end up being detected. But the ones that already exist in HTML will be detected as well. We're going to end up with duplicates in extracts, including duplicates of exported definitions such as the ruby element, which may be more problematic for Bikeshed. Nothing that should be blocking though?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

not blocking for sure; the reason I'm mentioning it is in case we had an annotation here that would help with that conflict down the line.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tracked in #1488. In the meantime, I'll warm up for a little Webref patching exercise.

"nightly": {
"sourcePath": "Overview.bs"
}
},
{
"url": "https://www.w3.org/TR/i18n-glossary/",
"categories": [
Expand Down
Loading