Skip to content

chore: Foundations pages, content, images added to DS website#367

Merged
Spark450 merged 5 commits into
GovAlta:alphafrom
mxsoco:mxsoco/foundation-content
May 27, 2025
Merged

chore: Foundations pages, content, images added to DS website#367
Spark450 merged 5 commits into
GovAlta:alphafrom
mxsoco:mxsoco/foundation-content

Conversation

@mxsoco
Copy link
Copy Markdown
Collaborator

@mxsoco mxsoco commented May 15, 2025

Addition of Foundations section, including pages, content, images.
Also, a change to the Color tokens JSON file.

image

@netlify
Copy link
Copy Markdown

netlify Bot commented May 15, 2025

Deploy Preview for abgov-ui-component-docs ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 9243403
🔍 Latest deploy log https://app.netlify.com/projects/abgov-ui-component-docs/deploys/6835f2324ac87b000845043d
😎 Deploy Preview https://deploy-preview-367--abgov-ui-component-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@mxsoco
Copy link
Copy Markdown
Collaborator Author

mxsoco commented May 15, 2025

@Spark450 @ArakTaiRoth New PR created for Foundations.

@twjeffery twjeffery self-requested a review May 27, 2025 15:43
@Spark450 Spark450 self-requested a review May 27, 2025 16:00
@Spark450 Spark450 self-assigned this May 27, 2025
Copy link
Copy Markdown
Collaborator

@Spark450 Spark450 left a comment

Choose a reason for hiding this comment

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

Hi Miguel, on second thought I think we should leave all instances of "Color" alone since this is aligned to the CSS value and our design tokens. Please disregard that feedback.

Comment thread src/routes/foundations/Accessibility.tsx
Comment thread src/routes/foundations/Accessibility.tsx
Comment thread src/routes/foundations/Accessibility.tsx
<h4>ARIA roles and headings</h4>
<p>The system uses ARIA roles to help assistive tools understand the purpose of different elements. These roles communicate what each element is for to screen readers and other assistive technologies.</p>
<ul>
<li><strong>Headings</strong> - Use structured headings (H1, H2, etc.) to organize content. H tags don't have to match the text size. For example, a Large Heading can be labeled as H1. Just keep headings consistent across pages, as this helps users find information quickly.</li>
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

suggest labelled here

Comment thread src/routes/foundations/Iconography.tsx Outdated
</GoabGrid>

<GoabSpacer vSpacing="l"></GoabSpacer>
<p>Explore core icons set and their properties <Link to="/components/icons">on the component page</Link>.</p>
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Explore the core icon set and its properties

Comment thread src/routes/foundations/Typography.tsx Outdated
<li><strong>Links</strong>
<ul>
<li>We offer three text sizes for links based on your design needs and context.</li>
<li>Links are styled with underlined text in interactive-default blue, for better accessibility and usability.</li>
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

remove comma on this line

Comment thread src/routes/foundations/Layout.tsx Outdated
</GoabContainer>

<h3 id="basic-form-template">Basic form layout example</h3>
<p>This layout is recommended for public-facing forms where the content needs to be simple, accessible, and often broken down into simple one question pages.</p>
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

one-question (add hyphen)

Comment thread src/routes/foundations/Layout.tsx Outdated

<h2 id="common-layouts">Common layouts</h2>
<h3 id="basic-form-layout">Basic form layout</h3>
<p>See <Link to="/patterns/simple-form">Public form pattern</Link> for more information</p>
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

add "." at the end of this line.

Comment thread src/routes/foundations/Layout.tsx Outdated
</GoabContainer>

<h3 id="side-nav-example">Side navigation layout example</h3>
<p>A 2-column layout is often used for internal applications, where there is a need for a side navigation.</p>
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

replace with "A 2-column layout is often used for internal applications where there is a need for side navigation."

Comment thread src/routes/foundations/Layout.tsx Outdated
<GoabDivider mt="m" mb="2xl"></GoabDivider>

<h3 id="side-nav-side-bar">Side navigation and side bar layout</h3>
<p>A 3-column layout that has a fixed side navigation as well as a second side panel on the right hand side. The third column is often used to show tertiary information, comments, or other documents alongside the main content area. Depending on your content, for large viewports it is recommended to use a maximum content width of 1176px on the main content area.</p>
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

change "right hand side" to "right side"

Copy link
Copy Markdown
Collaborator

@Spark450 Spark450 left a comment

Choose a reason for hiding this comment

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

Thanks for the changes! Looks good.

@Spark450 Spark450 merged commit 7cb49c0 into GovAlta:alpha May 27, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants