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

docs(manifest): Improvements to id page #36020

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

dipikabh
Copy link
Contributor

Description

This work is part of improving the web/manifest docs.

Apart from normalizing the page layout to include "Syntax" and "Values", this PR includes the following changes:

  • Organizes the information into a "Description" section and covers all info from the spec
  • Replaces the existing example with more realistic scenarios
  • Adds the tables of examples from the spec as a separate example section

Motivation

To ensure all sections have sufficient explanation, all caveats from spec are covered, and the pages follow a similar template

Additional details

Spec link: https://w3c.github.io/manifest/#id-member

Related issues and pull requests

Tracking issue: mdn/mdn#560

@dipikabh dipikabh requested a review from a team as a code owner September 23, 2024 20:05
@dipikabh dipikabh requested review from chrisdavidmills and removed request for a team September 23, 2024 20:05
@github-actions github-actions bot added Content:Manifest size/m [PR only] 51-500 LoC changed labels Sep 23, 2024
Copy link
Contributor

Preview URLs

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Content:Manifest size/m [PR only] 51-500 LoC changed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant