Skip to content

feat: add vertical stack variant to horizontal navigation item#3071

Open
martascorreia wants to merge 10 commits into
mainfrom
feat/vertical-stack-variant-to-horizontal-navigation-item
Open

feat: add vertical stack variant to horizontal navigation item#3071
martascorreia wants to merge 10 commits into
mainfrom
feat/vertical-stack-variant-to-horizontal-navigation-item

Conversation

@martascorreia

Copy link
Copy Markdown
Contributor

Description:

closes: #2990

Definition of Reviewable:

  • Documentation is created/updated
  • Migration Guide is created/updated
  • E2E tests (features, a11y, bug fixes) are created/updated
  • Stories (features, a11y) are created/updated
  • relevant tickets are linked

@martascorreia martascorreia changed the title Feat/vertical stack variant to horizontal navigation item feat: add vertical stack variant to horizontal navigation item Jun 8, 2026
@changeset-bot

changeset-bot Bot commented Jun 8, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 5c2cde5

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 4 packages
Name Type
@solid-design-system/components Minor
@solid-design-system/docs Minor
@solid-design-system/styles Minor
@solid-design-system/tokens Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@github-actions

github-actions Bot commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

🚀 Storybook has been deployed for branch feat_vertical-stack-variant-to-horizontal-navigation-item

@martascorreia martascorreia moved this from 📋 Backlog to 👀 In review in Solid Design System Project Board Jun 8, 2026
@martascorreia martascorreia self-assigned this Jun 8, 2026
<sd-icon name="system/image"></sd-icon>
<span>Navigation</span>
</sd-navigation-item>
<sd-navigation-item stacked disabled>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Remove disabled, current, current disabled here - we only show the stacked in default

@coraliefeil

Copy link
Copy Markdown
Contributor

Approved after comment above has been addressed. Thanks a lot. Good job @martascorreia <3

@coraliefeil coraliefeil removed their assignment Jun 11, 2026
@martascorreia

Copy link
Copy Markdown
Contributor Author

Approved after comment above has been addressed. Thanks a lot. Good job @martascorreia <3

Thank you, team effort! 😊

@tega-ekakitie

Copy link
Copy Markdown

@martascorreia Looks good to me

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: 👀 In review

Development

Successfully merging this pull request may close these issues.

feat[dev]: add vertical stack variant to horizontal navigation item

8 participants