-
Notifications
You must be signed in to change notification settings - Fork 1.4k
docs: Style macro reference table refactor #9329
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
base: main
Are you sure you want to change the base?
Conversation
|
Build successful! 🎉 |
|
Build successful! 🎉 |
| maxHeight: createSpectrumSizingProperty('maxHeight', (() => { | ||
| // auto is not a valid value for maxHeight | ||
| // eslint-disable-next-line @typescript-eslint/no-unused-vars | ||
| const {auto, ...rest} = height; | ||
| return {...rest, none: 'none'}; | ||
| })()), |
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.
note this change in the theme, should be pretty minor
| 'colors': { | ||
| description: 'Transition property for color related properties: color, background-color, border-color, fill, and stroke.' | ||
| }, | ||
| 'opacity': { | ||
| description: 'Transition property for opacity.' | ||
| }, | ||
| 'shadow': { | ||
| description: 'Transition property for box-shadow.' |
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.
Should transition be transitionProperty? The name actually matches the short hand, so is the intent here actually that people would use the transition shorthand and transition isn't actually a property?
|
Build successful! 🎉 |
|
Build successful! 🎉 |
|
Is |
Closes
✅ Pull Request Checklist:
📝 Test Instructions:
🧢 Your Project: