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

Move Web/API statics to _static URLs #26132

Closed
wants to merge 3 commits into from
Closed

Conversation

wbamberg
Copy link
Collaborator

Description

Motivation

Additional details

Related issues and pull requests

@github-actions github-actions bot added the Content:WebAPI Web API docs label Apr 12, 2023
@github-actions
Copy link
Contributor

github-actions bot commented Apr 12, 2023

Preview URLs (42 pages)
Flaws (20)

Note! 32 documents with no flaws that don't need to be listed. 🎉

URL: /en-US/docs/Web/API/RTCPeerConnection/generateCertificate_static
Title: RTCPeerConnection: generateCertificate() static method
Flaw count: 4

  • macros:
    • /en-US/docs/Web/API/AlgorithmIdentifier does not exist
    • /en-US/docs/Web/API/Algorithm does not exist
    • /en-US/docs/Web/API/RTCCertificateExpiration does not exist
    • /en-US/docs/Web/API/Algorithm does not exist

URL: /en-US/docs/Web/API/MouseEvent/WEBKIT_FORCE_AT_MOUSE_DOWN_static
Title: MouseEvent: WEBKIT_FORCE_AT_MOUSE_DOWN static property
Flaw count: 1

  • macros:
    • /en-US/docs/Web/API/MouseEvent/WEBKIT_FORCE_AT_FORCE_MOUSE_DOWN redirects to /en-US/docs/Web/API/MouseEvent/WEBKIT_FORCE_AT_FORCE_MOUSE_DOWN_static

URL: /en-US/docs/Web/API/MouseEvent/WEBKIT_FORCE_AT_FORCE_MOUSE_DOWN_static
Title: MouseEvent: WEBKIT_FORCE_AT_FORCE_MOUSE_DOWN static property
Flaw count: 1

  • macros:
    • /en-US/docs/Web/API/MouseEvent/WEBKIT_FORCE_AT_MOUSE_DOWN redirects to /en-US/docs/Web/API/MouseEvent/WEBKIT_FORCE_AT_MOUSE_DOWN_static

URL: /en-US/docs/Web/API/MediaSource/isTypeSupported_static
Title: MediaSource: isTypeSupported() static method
Flaw count: 1

  • macros:
    • wrong xref macro used (consider changing which macro you use)

URL: /en-US/docs/Web/API/MediaSource/canConstructInDedicatedWorker_static
Title: MediaSource: canConstructInDedicatedWorker static property
Flaw count: 1

  • macros:
    • wrong xref macro used (consider changing which macro you use)

URL: /en-US/docs/Web/API/RTCRtpReceiver/getCapabilities_static
Title: RTCRtpReceiver: getCapabilities() static method
Flaw count: 1

  • macros:
    • /en-US/docs/Web/API/RTCRtpSender/getCapabilities redirects to /en-US/docs/Web/API/RTCRtpSender/getCapabilities_static

URL: /en-US/docs/Web/API/RTCRtpSender/getCapabilities_static
Title: RTCRtpSender: getCapabilities() static method
Flaw count: 1

  • macros:
    • /en-US/docs/Web/API/RTCRtpReceiver/getCapabilities redirects to /en-US/docs/Web/API/RTCRtpReceiver/getCapabilities_static

URL: /en-US/docs/Web/API/URL/createObjectURL_static
Title: URL: createObjectURL() static method
Flaw count: 4

  • macros:
    • /en-US/docs/Web/API/URL/revokeObjectURL redirects to /en-US/docs/Web/API/URL/revokeObjectURL_static
    • /en-US/docs/Web/API/URL/revokeObjectURL redirects to /en-US/docs/Web/API/URL/revokeObjectURL_static
    • /en-US/docs/Web/API/URL/createObjectURL redirects to /en-US/docs/Web/API/URL/createObjectURL_static
    • /en-US/docs/Web/API/URL/revokeObjectURL redirects to /en-US/docs/Web/API/URL/revokeObjectURL_static

URL: /en-US/docs/Web/API/URL/revokeObjectURL_static
Title: URL: revokeObjectURL() static method
Flaw count: 3

  • macros:
    • /en-US/docs/Web/API/URL/createObjectURL redirects to /en-US/docs/Web/API/URL/createObjectURL_static
    • /en-US/docs/Web/API/URL/createObjectURL redirects to /en-US/docs/Web/API/URL/createObjectURL_static
    • /en-US/docs/Web/API/URL/createObjectURL redirects to /en-US/docs/Web/API/URL/createObjectURL_static

URL: /en-US/docs/Web/API/CSS/RegisterProperty_static
Title: CSS: registerProperty() static method
Flaw count: 3

  • macros:
    • /en-US/docs/Web/API/CSS/supports redirects to /en-US/docs/Web/API/CSS/supports_static
    • /en-US/docs/Web/API/CSS/escape redirects to /en-US/docs/Web/API/CSS/escape_static
    • /en-US/docs/Web/API/CSS/factory_functions redirects to /en-US/docs/Web/API/CSS/factory_functions_static

(comment last updated: 2023-04-12 03:06:03)

@OnkarRuikar
Copy link
Contributor

OnkarRuikar commented Apr 12, 2023

@teoli2003 we definitely have to merge it before CSS content restructuring.


About the front-matter attribute order change, let me know if we've decided/finalized on this order. Is it documented somewhere in meta docs or discussions?

Comment on lines 2 to 6
title: 'Notification: requestPermission() static method'
slug: Web/API/Notification/requestPermission_static
page-type: web-api-static-method
browser-compat: api.Notification.requestPermission
short-title: requestPermission()
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Suggested change
title: 'Notification: requestPermission() static method'
slug: Web/API/Notification/requestPermission_static
page-type: web-api-static-method
browser-compat: api.Notification.requestPermission
short-title: requestPermission()
title: 'Notification: requestPermission() static method'
short-title: requestPermission()
slug: Web/API/Notification/requestPermission_static
page-type: web-api-static-method
browser-compat: api.Notification.requestPermission

Comment on lines 2 to 6
title: 'PerformanceObserver: supportedEntryTypes static property'
slug: Web/API/PerformanceObserver/supportedEntryTypes_static
page-type: web-api-static-property
browser-compat: api.PerformanceObserver.supportedEntryTypes
short-title: supportedEntryTypes
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Suggested change
title: 'PerformanceObserver: supportedEntryTypes static property'
slug: Web/API/PerformanceObserver/supportedEntryTypes_static
page-type: web-api-static-property
browser-compat: api.PerformanceObserver.supportedEntryTypes
short-title: supportedEntryTypes
title: 'PerformanceObserver: supportedEntryTypes static property'
short-title: supportedEntryTypes
slug: Web/API/PerformanceObserver/supportedEntryTypes_static
page-type: web-api-static-property
browser-compat: api.PerformanceObserver.supportedEntryTypes

Comment on lines 2 to 9
title: >-
PublicKeyCredential: isUserVerifyingPlatformAuthenticatorAvailable() static
method
slug: >-
Web/API/PublicKeyCredential/isUserVerifyingPlatformAuthenticatorAvailable_static
page-type: web-api-static-method
browser-compat: api.PublicKeyCredential.isUserVerifyingPlatformAuthenticatorAvailable
short-title: isUserVerifyingPlatformAuthenticatorAvailable()
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Suggested change
title: >-
PublicKeyCredential: isUserVerifyingPlatformAuthenticatorAvailable() static
method
slug: >-
Web/API/PublicKeyCredential/isUserVerifyingPlatformAuthenticatorAvailable_static
page-type: web-api-static-method
browser-compat: api.PublicKeyCredential.isUserVerifyingPlatformAuthenticatorAvailable
short-title: isUserVerifyingPlatformAuthenticatorAvailable()
title: >-
PublicKeyCredential: isUserVerifyingPlatformAuthenticatorAvailable() static
method
short-title: isUserVerifyingPlatformAuthenticatorAvailable()
slug: >-
Web/API/PublicKeyCredential/isUserVerifyingPlatformAuthenticatorAvailable_static
page-type: web-api-static-method
browser-compat: api.PublicKeyCredential.isUserVerifyingPlatformAuthenticatorAvailable

Comment on lines 2 to 6
title: 'Response: error() static method'
slug: Web/API/Response/error_static
page-type: web-api-static-method
browser-compat: api.Response.error
short-title: error()
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Suggested change
title: 'Response: error() static method'
slug: Web/API/Response/error_static
page-type: web-api-static-method
browser-compat: api.Response.error
short-title: error()
title: 'Response: error() static method'
short-title: error()
slug: Web/API/Response/error_static
page-type: web-api-static-method
browser-compat: api.Response.error

Comment on lines 2 to 6
title: 'Response: redirect() static method'
slug: Web/API/Response/redirect_static
page-type: web-api-static-method
browser-compat: api.Response.redirect
short-title: redirect()
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Suggested change
title: 'Response: redirect() static method'
slug: Web/API/Response/redirect_static
page-type: web-api-static-method
browser-compat: api.Response.redirect
short-title: redirect()
title: 'Response: redirect() static method'
short-title: redirect()
slug: Web/API/Response/redirect_static
page-type: web-api-static-method
browser-compat: api.Response.redirect

Comment on lines 2 to 6
title: 'RTCPeerConnection: generateCertificate() static method'
slug: Web/API/RTCPeerConnection/generateCertificate_static
page-type: web-api-static-method
browser-compat: api.RTCPeerConnection.generateCertificate
short-title: generateCertificate()
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Suggested change
title: 'RTCPeerConnection: generateCertificate() static method'
slug: Web/API/RTCPeerConnection/generateCertificate_static
page-type: web-api-static-method
browser-compat: api.RTCPeerConnection.generateCertificate
short-title: generateCertificate()
title: 'RTCPeerConnection: generateCertificate() static method'
short-title: generateCertificate()
slug: Web/API/RTCPeerConnection/generateCertificate_static
page-type: web-api-static-method
browser-compat: api.RTCPeerConnection.generateCertificate

Comment on lines 2 to 6
title: 'RTCRtpReceiver: getCapabilities() static method'
slug: Web/API/RTCRtpReceiver/getCapabilities_static
page-type: web-api-static-method
browser-compat: api.RTCRtpReceiver.getCapabilities
short-title: getCapabilities()
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Suggested change
title: 'RTCRtpReceiver: getCapabilities() static method'
slug: Web/API/RTCRtpReceiver/getCapabilities_static
page-type: web-api-static-method
browser-compat: api.RTCRtpReceiver.getCapabilities
short-title: getCapabilities()
title: 'RTCRtpReceiver: getCapabilities() static method'
short-title: getCapabilities()
slug: Web/API/RTCRtpReceiver/getCapabilities_static
page-type: web-api-static-method
browser-compat: api.RTCRtpReceiver.getCapabilities

Comment on lines 2 to 6
title: 'RTCRtpSender: getCapabilities() static method'
slug: Web/API/RTCRtpSender/getCapabilities_static
page-type: web-api-static-method
browser-compat: api.RTCRtpSender.getCapabilities
short-title: getCapabilities()
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Suggested change
title: 'RTCRtpSender: getCapabilities() static method'
slug: Web/API/RTCRtpSender/getCapabilities_static
page-type: web-api-static-method
browser-compat: api.RTCRtpSender.getCapabilities
short-title: getCapabilities()
title: 'RTCRtpSender: getCapabilities() static method'
short-title: getCapabilities()
slug: Web/API/RTCRtpSender/getCapabilities_static
page-type: web-api-static-method
browser-compat: api.RTCRtpSender.getCapabilities

Comment on lines 2 to 6
title: 'URL: createObjectURL() static method'
slug: Web/API/URL/createObjectURL_static
page-type: web-api-static-method
browser-compat: api.URL.createObjectURL
short-title: createObjectURL()
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Suggested change
title: 'URL: createObjectURL() static method'
slug: Web/API/URL/createObjectURL_static
page-type: web-api-static-method
browser-compat: api.URL.createObjectURL
short-title: createObjectURL()
title: 'URL: createObjectURL() static method'
short-title: createObjectURL()
slug: Web/API/URL/createObjectURL_static
page-type: web-api-static-method
browser-compat: api.URL.createObjectURL

Comment on lines 2 to 6
title: 'URL: revokeObjectURL() static method'
slug: Web/API/URL/revokeObjectURL_static
page-type: web-api-static-method
browser-compat: api.URL.revokeObjectURL
short-title: revokeObjectURL()
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Suggested change
title: 'URL: revokeObjectURL() static method'
slug: Web/API/URL/revokeObjectURL_static
page-type: web-api-static-method
browser-compat: api.URL.revokeObjectURL
short-title: revokeObjectURL()
title: 'URL: revokeObjectURL() static method'
short-title: revokeObjectURL()
slug: Web/API/URL/revokeObjectURL_static
page-type: web-api-static-method
browser-compat: api.URL.revokeObjectURL

Comment on lines 2 to 6
title: 'VideoDecoder: isConfigSupported() static method'
slug: Web/API/VideoDecoder/isConfigSupported_static
page-type: web-api-static-method
browser-compat: api.VideoDecoder.isConfigSupported
short-title: isConfigSupported()
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Suggested change
title: 'VideoDecoder: isConfigSupported() static method'
slug: Web/API/VideoDecoder/isConfigSupported_static
page-type: web-api-static-method
browser-compat: api.VideoDecoder.isConfigSupported
short-title: isConfigSupported()
title: 'VideoDecoder: isConfigSupported() static method'
short-title: isConfigSupported()
slug: Web/API/VideoDecoder/isConfigSupported_static
page-type: web-api-static-method
browser-compat: api.VideoDecoder.isConfigSupported

Comment on lines 2 to 6
title: 'VideoEncoder: isConfigSupported() static method'
slug: Web/API/VideoEncoder/isConfigSupported_static
page-type: web-api-static-method
browser-compat: api.VideoEncoder.isConfigSupported
short-title: isConfigSupported()
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Suggested change
title: 'VideoEncoder: isConfigSupported() static method'
slug: Web/API/VideoEncoder/isConfigSupported_static
page-type: web-api-static-method
browser-compat: api.VideoEncoder.isConfigSupported
short-title: isConfigSupported()
title: 'VideoEncoder: isConfigSupported() static method'
short-title: isConfigSupported()
slug: Web/API/VideoEncoder/isConfigSupported_static
page-type: web-api-static-method
browser-compat: api.VideoEncoder.isConfigSupported

Comment on lines 2 to 8
title: 'XRWebGLLayer: getNativeFramebufferScaleFactor() static method'
slug: Web/API/XRWebGLLayer/getNativeFramebufferScaleFactor_static
page-type: web-api-static-method
browser-compat: api.XRWebGLLayer.getNativeFramebufferScaleFactor
short-title: getNativeFramebufferScaleFactor()
status:
- experimental
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Suggested change
title: 'XRWebGLLayer: getNativeFramebufferScaleFactor() static method'
slug: Web/API/XRWebGLLayer/getNativeFramebufferScaleFactor_static
page-type: web-api-static-method
browser-compat: api.XRWebGLLayer.getNativeFramebufferScaleFactor
short-title: getNativeFramebufferScaleFactor()
status:
- experimental
title: 'XRWebGLLayer: getNativeFramebufferScaleFactor() static method'
short-title: getNativeFramebufferScaleFactor()
slug: Web/API/XRWebGLLayer/getNativeFramebufferScaleFactor_static
page-type: web-api-static-method
status:
- experimental
browser-compat: api.XRWebGLLayer.getNativeFramebufferScaleFactor

@OnkarRuikar
Copy link
Contributor

If possible we do not have to change double quotes " in the title considering double(~5514) vs single(~147) in the repo and #26028 (comment).

@wbamberg
Copy link
Collaborator Author

This isn't merged yet! And I'd be happy to block it for a while if that one is likely to get in any time soon. I'm not totally sure it's what we want to do (but think it probably is).

I also think we should basically deprecate domxref calls like {{domxref("Interface.member")}}, and encourage people to do:

{{domxref("Interface/member")} for non-static
{{domxref("Interface/member_static")} for non-static

...and have domxref cook up sensible link text (unless param 2 is given).

@teoli2003 we definitely have to merge it before CSS content restructuring.

About the front-matter attribute order change, let me know if we've decided/finalized on this order. Is it documented somewhere in meta docs or discussions?

No, we should though. But it makes sense (to me) to have short-title right after title. I think this is connected to mdn/yari#7825 (comment).

@wbamberg
Copy link
Collaborator Author

If possible we do not have to change double quotes " in the title considering double(~5514) vs single(~147) in the repo and #26028 (comment).

I would love that but this is what yarn content move does 🙄 . So it would be great to get that fixed.

@github-actions github-actions bot added the merge conflicts 🚧 [PR only] label Apr 12, 2023
@github-actions
Copy link
Contributor

This pull request has merge conflicts that must be resolved before it can be merged.

@teoli2003
Copy link
Member

About the front-matter attribute order change, let me know if we've decided/finalized on this order. Is it documented somewhere in meta docs or discussions?

No, we should though. But it makes sense (to me) to have short-title right after title. I think this is connected to mdn/yari#7825 (comment).

We should document the order and enforce it with linting. We have many YAML headers nowadays. Does anybody know a good YAML linter?

@OnkarRuikar
Copy link
Contributor

We should document the order and enforce it with linting. We have many YAML headers nowadays. Does anybody know a good YAML linter?

Do we need a GitHub discussion to fix the order of front-matter attributes?

@OnkarRuikar
Copy link
Contributor

If possible we do not have to change double quotes " in the title considering double(~5514) vs single(~147) in the repo and #26028 (comment).

I would love that but this is what yarn content move does roll_eyes . So it would be great to get that fixed.

@wbamberg
Copy link
Collaborator Author

Closing in favour of #26482.

@wbamberg wbamberg closed this Apr 28, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 1, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants