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

Move level 3 nav items to page subnav #10509

Merged
merged 1 commit into from
Sep 18, 2023
Merged

Move level 3 nav items to page subnav #10509

merged 1 commit into from
Sep 18, 2023

Conversation

sam-b-rose
Copy link
Member

@sam-b-rose sam-b-rose commented Sep 18, 2023

Closes #10151

Creates a Subnav component that will inject subnav items into the MDX. We could also probably add this to the Lede component, but it is currently a standalone component.

Note

The styles aren't updated, but are scaffolded to be quickly updated with the Uplift colors and Figma spec

<Lede>{title}</Ledel>

<Subnav />

Screenshot example using Typography

subnav-proto

Copy link
Contributor

@yurm04 yurm04 left a comment

Choose a reason for hiding this comment

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

Looks great and works event better ✨

the /desgin/typography/index.md page will also need the Subnav component added to the markdown. Feel free to add before or after merging if you please. Only one other comment that is not a blocker

Awesome job, Sam!!

@tjonx tjonx self-requested a review September 18, 2023 16:30
Copy link
Contributor

@tjonx tjonx left a comment

Choose a reason for hiding this comment

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

Super happy that we can get this L3 nav!! 🥳

@sam-b-rose sam-b-rose merged commit de5b7fa into next Sep 18, 2023
7 checks passed
@sam-b-rose sam-b-rose deleted the 10151-docs-subnav branch September 18, 2023 17:35
sophschneider pushed a commit that referenced this pull request Sep 19, 2023
Closes #10151

Creates a `Subnav` component that will inject subnav items into the MDX.
We could also probably add this to the `Lede` component, but it is
currently a standalone component.

> [!Note]
> The styles aren't updated, but are scaffolded to be quickly updated
with the Uplift colors and Figma spec

```mdx

<Lede>{title}</Ledel>

<Subnav />

```

### Screenshot example using Typography

<img width="1471" alt="subnav-proto"
src="https://github.com/Shopify/polaris/assets/11774595/270a00d8-15f9-484b-a5d5-de8fd162eb6f">
AnnaCheba pushed a commit to AnnaCheba/polaris that referenced this pull request Apr 22, 2024
Closes Shopify#10151

Creates a `Subnav` component that will inject subnav items into the MDX.
We could also probably add this to the `Lede` component, but it is
currently a standalone component.

> [!Note]
> The styles aren't updated, but are scaffolded to be quickly updated
with the Uplift colors and Figma spec

```mdx

<Lede>{title}</Ledel>

<Subnav />

```

### Screenshot example using Typography

<img width="1471" alt="subnav-proto"
src="https://github.com/Shopify/polaris/assets/11774595/270a00d8-15f9-484b-a5d5-de8fd162eb6f">
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.

3 participants