-
Notifications
You must be signed in to change notification settings - Fork 638
Remove sx and BoxWithFallback from the Spinner
component
#6708
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
Conversation
🦋 Changeset detectedLatest commit: 1cc55d6 The changes in this PR will be included in the next version bump. This PR includes changesets to release 2 packages
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 |
👋 Hi, this pull request contains changes to the source code that github/github depends on. If you are GitHub staff, we recommend testing these changes with github/github using the integration workflow. Thanks! |
size-limit report 📦
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR removes the deprecated sx
prop and BoxWithFallback
wrapper from the Spinner
component as part of ongoing efforts to simplify the component API and reduce dependencies on the styled-system approach.
- Removes
sx
prop andSxProp
type from the mainSpinner
component - Eliminates
BoxWithFallback
wrapper andStyledSpinner
function in favor of direct component usage - Maintains backward compatibility by adding a styled wrapper in the legacy
styled-react
package
Reviewed Changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated no comments.
Show a summary per file
File | Description |
---|---|
packages/react/src/Spinner/Spinner.tsx | Removes sx prop support and BoxWithFallback wrapper, simplifies component implementation |
packages/styled-react/src/index.tsx | Adds backward-compatible Spinner wrapper with sx prop support for legacy usage |
packages/react/src/Spinner/Spinner.docs.json | Updates component documentation to remove sx prop reference |
packages/react/src/Spinner/Spinner.dev.stories.tsx | Updates dev story to use CSS class instead of sx prop |
packages/react/src/Spinner/Spinner.dev.stories.module.css | Adds CSS module for dev story styling |
.changeset/beige-shrimps-sleep.md | Documents breaking change for major version release |
👋 Hi, there are new commits since the last successful integration test. We recommend running the integration workflow once more, unless you are sure the new changes do not affect github/github. Thanks! |
👋 Hi from github/github-ui! Your integration PR is ready: https://github.com/github/github-ui/pull/3048 |
🟢 ci completed with status |
Remove
sx
andBoxWithFallback
from theSpinner
componentCloses #5819
sx usage, tracking here: https://github.com/github/primer/issues/5827
Rollout strategy