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

Group Manifest docs into references #37850

Merged
merged 8 commits into from
Jan 31, 2025
Merged

Group Manifest docs into references #37850

merged 8 commits into from
Jan 31, 2025

Conversation

pepelsbey
Copy link
Member

@pepelsbey pepelsbey commented Jan 28, 2025

Moves pages into the reference folder

- web/manifest/background_color
+ web/manifest/reference/background_color
etc.

Creates landing pages for references

+ web/manifest/reference/index.md

Adjusts sidebars

  • Adds /Reference to the {{QuickLinksWithSubpages}} macro to avoid nesting

Uses new page type

+ page-type: listing-page

Motivation

Docs IA & maintenance

TODO

  • Check links in BCD
  • Fix links in web-features

@pepelsbey pepelsbey requested a review from bsmth January 28, 2025 14:45
@pepelsbey pepelsbey requested review from a team as code owners January 28, 2025 14:45
@github-actions github-actions bot added Content:CSS Cascading Style Sheets docs Content:WebAPI Web API docs Content:Glossary Glossary entries Content:Meta Content in the meta docs Content:Manifest Content:PWA Progressive Web Apps content system [PR only] Infrastructure and configuration for the project labels Jan 28, 2025
@pepelsbey pepelsbey removed request for a team January 28, 2025 14:45
@github-actions github-actions bot added the size/m [PR only] 51-500 LoC changed label Jan 28, 2025
Copy link
Contributor

github-actions bot commented Jan 28, 2025

Preview URLs (52 pages)
Flaws (1)

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

URL: /en-US/docs/Web/CSS/env
Title: env()
Flaw count: 1

  • broken_links:
    • Can't resolve /en-US/docs/Web/CSS/declaration-value
External URLs (1)

URL: /en-US/docs/Web/Progressive_web_apps/How_to/Share_data_between_apps
Title: Share data between apps

(comment last updated: 2025-01-31 11:25:43)

@github-actions github-actions bot removed the system [PR only] Infrastructure and configuration for the project label Jan 30, 2025
Copy link
Member

@bsmth bsmth left a comment

Choose a reason for hiding this comment

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

TY!

@Rumyra Rumyra merged commit ab4090c into mdn:main Jan 31, 2025
9 checks passed
@pepelsbey pepelsbey deleted the move/manifest branch January 31, 2025 14:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Content:CSS Cascading Style Sheets docs Content:Glossary Glossary entries Content:Manifest Content:Meta Content in the meta docs Content:PWA Progressive Web Apps content Content:WebAPI Web API docs size/m [PR only] 51-500 LoC changed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants