Conversation
fb8ec53 to
28729e1
Compare
28729e1 to
90f8567
Compare
✅ Deploy Preview for goa-design-2 ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Spark450
left a comment
There was a problem hiding this comment.
This review includes the following so far:
- Overview
- Accessibility
- Brand guidelines
- Style guide -> Colour
- Style guide -> Iconography
Spark450
left a comment
There was a problem hiding this comment.
A few more comments, still working through the remainder of the pages.
Spark450
left a comment
There was a problem hiding this comment.
Okay these are the last of them 😅
90f8567 to
b106bf8
Compare
|
@Spark450 All the changes you requested have been made |
b106bf8 to
34f4da0
Compare
Spark450
left a comment
There was a problem hiding this comment.
Just a few comments on Layout, Illustration looks good!
34f4da0 to
8df4423
Compare
|
The changes look good @ArakTaiRoth approved. Lets get this out of draft so we can release what we have. |
There was a problem hiding this comment.
Pull request overview
Adds the requested /foundations documentation section to the Astro docs site, including new style-guide and content-guideline pages and updates to navigation so Foundations appears in the site menu hierarchy.
Changes:
- Added multiple new Foundations pages (Design at GoA, Accessibility, Brand guidelines, Style guide pages, Content guidelines pages).
- Updated docs navigation to include Foundations as a top-level section with a dedicated submenu and grouped pages.
- Added shared UI helpers for Foundations pages (support callout, static grid wrapper) and adjusted
DoDontbehavior for static guidance content.
Reviewed changes
Copilot reviewed 20 out of 37 changed files in this pull request and generated 6 comments.
Show a summary per file
| File | Description |
|---|---|
| docs/src/pages/foundations/index.astro | Updates Foundations landing page content and adds support callout. |
| docs/src/pages/foundations/accessibility.astro | Adds Accessibility foundations page. |
| docs/src/pages/foundations/brand-guidelines.astro | Adds Brand guidelines foundations page. |
| docs/src/pages/foundations/style-guide/colour.astro | Adds Colour style guide page with token lists. |
| docs/src/pages/foundations/style-guide/iconography.astro | Adds Iconography style guide page and icon-size table. |
| docs/src/pages/foundations/style-guide/illustration.astro | Adds Illustration style guide page with token lists and process guidance. |
| docs/src/pages/foundations/style-guide/layout.astro | Adds Layout style guide page. |
| docs/src/pages/foundations/style-guide/logo.astro | Adds Logo style guide page and related images. |
| docs/src/pages/foundations/style-guide/photography.astro | Adds Photography style guide page. |
| docs/src/pages/foundations/style-guide/typography.astro | Adds Typography style guide page. |
| docs/src/pages/foundations/content-guidelines/capitalization.astro | Adds Capitalization content guideline page. |
| docs/src/pages/foundations/content-guidelines/helper-text.astro | Adds Helper text content guideline page with Do/Dont previews. |
| docs/src/components/nav/ParentMenu.tsx | Adds Foundations to top-level nav sections and tightens types. |
| docs/src/components/nav/FoundationsSubMenu.tsx | Adds Foundations submenu pages/groups and navigation handling. |
| docs/src/components/SiteNav.tsx | Updates documentation/comments for submenu behavior including Foundations. |
| docs/src/components/FoundationsSupportCallout.astro | Adds reusable “Need help…” callout used across Foundations pages. |
| docs/src/components/StaticGuidanceGrid.astro | Adds shared responsive grid wrapper for static Do/Dont layouts. |
| docs/src/components/DoDont.astro | Adjusts wrapper logic for static pages and adds preview sizing handling. |
| docs/public/images/foundations/** | Adds new images used by Foundations pages. |
3cf9a61 to
068cd43
Compare
bdfranck
left a comment
There was a problem hiding this comment.
I reviewed the of the completed pages...
- I fixed a couple minor grammar issues
- I fixed some issues flagged by Copilot
- I removed all duplicate icons in the "Extended icon set" section
The result looks good to me! 👍
Spark450
left a comment
There was a problem hiding this comment.
Thanks for taking care of those comments @bdfranck
Can you do a quick review @chrisolsen
twjeffery
left a comment
There was a problem hiding this comment.
A few issues:
- Foundations should be at the bottom of the nav items (last, under Tokens).
- Foundations should have a secondary side nav like the rest of the main sections (like Components and Get started) so that there is not multiple levels of nesting.
- (BUG) Right now when you click a page in foundations it goes to that page but changes the secondary side menu for foundations (that I reference above).
- These need to be consolidated, and all foundations pages should use this secondary menu.
Screen.Recording.2026-03-30.at.6.04.33.PM.mov
- Remove the icon on the style guide and content guidelines groups.
@twjeffery Regarding your issue #2 and #3. Those changes are already in place if you pull this down, build it, and run it locally you'll see. For some reason the netlify deploy never got updated after the initial creation of this PR. |
Ah ok, awesome, good to know. Thanks |
|
🎉 This PR is included in version 2.0.0-dev.3 🎉 The release is available on: Your semantic-release bot 📦🚀 |
|
🎉 This PR is included in version 2.0.0-dev.2 🎉 The release is available on: Your semantic-release bot 📦🚀 |
|
🎉 This PR is included in version 7.0.0-dev.2 🎉 The release is available on: Your semantic-release bot 📦🚀 |
|
🎉 This PR is included in version 5.0.0-dev.2 🎉 The release is available on: Your semantic-release bot 📦🚀 |
|
🎉 This PR is included in version 2.0.0-next.1 🎉 The release is available on: Your semantic-release bot 📦🚀 |
|
🎉 This PR is included in version 2.0.0-next.1 🎉 The release is available on: Your semantic-release bot 📦🚀 |
|
🎉 This PR is included in version 7.0.0-next.1 🎉 The release is available on: Your semantic-release bot 📦🚀 |
|
🎉 This PR is included in version 5.0.0-next.1 🎉 The release is available on: Your semantic-release bot 📦🚀 |
|
🎉 This PR is included in version 2.0.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
|
🎉 This PR is included in version 2.0.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
|
🎉 This PR is included in version 7.0.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
|
🎉 This PR is included in version 5.0.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |


Before (the change)
After (the change)
This adds the Foundation pages requested in #3080 . The content for these pages is found here and in the pages linked to below this page.
All pages are located at
/foundationsRefer to the story for what pages are done and reviewable, those are checked off