-
Notifications
You must be signed in to change notification settings - Fork 22.5k
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
Add Web/API page types, part 13 #16837
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One error I think.
Good catch, for the 4 static methods on IDBKeyRange
. There are no mention of it in the articles. We probably want, once we have page types all over APIs, to be sure static methods are documented (in the prose) as such.
@@ -1,6 +1,7 @@ | |||
--- | |||
title: indexedDB | |||
slug: Web/API/indexedDB | |||
page-type: web-api-interface |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is a global property, not an interface.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ooh, good catch, thank you!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
and yes, at the moment we don't have any reliable way to indicate statics AFAIK.
See #16255. This is part 13 of a series of PRs to add page types to the Web/API documentation.