Skip to content

feat(MenuLink): bold current-state, Storybook controls voor iconen/badge, RTL + LongText stories#134

Merged
jeffreylauwers merged 1 commit intomainfrom
feature/menu-link
Apr 4, 2026
Merged

feat(MenuLink): bold current-state, Storybook controls voor iconen/badge, RTL + LongText stories#134
jeffreylauwers merged 1 commit intomainfrom
feature/menu-link

Conversation

@jeffreylauwers
Copy link
Copy Markdown
Owner

Summary

  • Bold current state — nieuw dsn.menu-link.current.font-weight token (→ {dsn.text.font-weight.bold}) en toegepast in CSS via font-weight: var(--dsn-menu-link-current-font-weight)
  • Storybook controlsiconStart, iconEnd en numberBadge zijn nu selecteerbaar in het Controls-paneel (zelfde icon-lijst als Link/Button; badge met 4 presets)
  • Nieuwe storiesWithIconEnd, LongText, RTL, RTLLongText
  • Losse stories — gebruik nu renderSingle-helper zodat elk individueel story één enkel MenuLink toont zonder geneste lijsten; consistenter met andere component-stories

Test plan

  • Storybook: Current story toont vetgedrukte tekst
  • Storybook: Controls-paneel bevat iconStart, iconEnd en numberBadge selects
  • Storybook: WithIconEnd story rendert correct
  • Storybook: RTL story spiegelt correct (border-inline-start rechts, iconen geflipped)
  • Storybook: LongText story toont tekstomloop
  • Alle 1213 tests groen (pnpm test)
  • TypeScript schoon (pnpm --filter storybook exec tsc --noEmit)

🤖 Generated with Claude Code

…trols, RTL + LongText stories

- Voegt `dsn.menu-link.current.font-weight` token toe (→ bold) en past het toe in CSS
- Voegt iconStart, iconEnd en numberBadge argTypes toe in Storybook met select-controls
- Voegt WithIconEnd, LongText, RTL en RTLLongText stories toe
- Trekt losse stories gelijk met andere componenten via renderSingle-helper (geen geneste lijsten)
- Breidt AllStates uit met iconEnd-variant

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@jeffreylauwers jeffreylauwers merged commit 50721d9 into main Apr 4, 2026
3 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.

1 participant