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 app store members #36605

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

dipikabh
Copy link
Contributor

@dipikabh dipikabh commented Nov 1, 2024

Description

This PR is part of improving the web/manifest docs and updates the categories, screenshots, and description pages:

  • Added "Syntax", "Values", and "Description" sections
  • Covered caveats from the spec
  • Added prose to the examples
  • Removed dir and lang from description page because they are not implemented

(Also fixed formatting in icons Syntax block)

Motivation

To ensure all sections have sufficient explanation, all info from spec is covered, and the manifest pages follow a similar template

Additional details

Spec: https://w3c.github.io/manifest-app-info/

Related issues and pull requests

Tracking issue: mdn/mdn#560

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

github-actions bot commented Nov 1, 2024

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