docs/readme index tweak#24
Merged
Merged
Conversation
Owner
marklearst
commented
Jan 8, 2026
marklearst
commented
- chore(env): refresh node tooling
- chore(build): modernize build configs
- chore(tailwind): remove legacy configs
- chore(deps): update modern stack
- chore(ci): refresh workflows
- feat(storybook): modernize core config
- feat(styles): add foundation token stories
- feat(form-field): normalize stories and helpers
- chore(icons): refresh generated assets
- feat(navigation): align stories and structure
- feat(menu): refresh stories and popover
- feat(table): refresh stories and helpers
- feat(components): refresh core controls
- feat(layout): refresh shell components
- feat(components): refresh misc primitives
- docs(home): refresh docs and utilities
- feat(listbox): add multiple support
- fix(storybook): align story args
- fix: listbox stories and docs tweaks
Align listbox story args and update README/index copy.
There was a problem hiding this comment.
Pull request overview
This PR modernizes the codebase by refreshing tooling, build configurations, and component implementations. The changes primarily involve formatting updates (switching from double to single quotes, removing semicolons), migrating from deprecated libraries (react-popper to @floating-ui/react), improving type imports, and adding comprehensive test coverage across components.
- Standardized code style (single quotes, no semicolons)
- Migrated from
react-popperto@floating-ui/reactfor positioning - Updated Storybook imports from
@storybook/reactto@storybook/react-vite - Added test files for previously untested components
Reviewed changes
Copilot reviewed 297 out of 886 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
src/components/tooltip/tooltip.tsx |
Migrated from usePopper to useFloating for positioning logic |
src/components/popover-menu/popover-menu.tsx |
Updated positioning from react-popper to @floating-ui/react |
src/components/navigation/navigation-popover.tsx |
Replaced usePopper with useFloating hook |
src/components/slot/slot.tsx |
Fixed duplicate className in props merge logic |
src/components/spinner/spinner.tsx |
Changed hardcoded color values to CSS variables |
src/components/link/link.tsx |
Fixed className merging to preserve child element classes |
| Test files (multiple) | Added comprehensive unit tests for components |
| Story files (multiple) | Enhanced interactivity with dynamic args and controls |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.