Closed shenjunjian closed 3 weeks ago
The changes in this pull request enhance the button component's styling capabilities by introducing new variables for icon sizes in the index.less
and vars.less
files. Specifically, a new variable @iconSize
is added to the button's size mixin, allowing dynamic sizing of icons based on button size. Additionally, several CSS custom properties are defined for different button sizes, ensuring consistent icon dimensions across various button types. The overall structure of the existing styles remains unchanged.
File Path | Change Summary |
---|---|
packages/theme/src/button/index.less | Added variable @iconSize to .size-mixin(@size) for dynamic icon sizing in .is-only-icon state. |
packages/theme/src/button/vars.less | Introduced new CSS custom properties for button icon sizes: --tv-Button-size-icon-font-size , --tv-Button-size-icon-font-size-large , --tv-Button-size-icon-font-size-medium , --tv-Button-size-icon-font-size-small , and --tv-Button-size-icon-font-size-mini . |
index.less
and vars.less
, directly related to button styling and icon sizing.vars.less
and index.less
, relevant to overall button styling enhancements.index.less
, potentially relevant to icon alignment and sizing changes.bug
In the garden where buttons bloom,
Icons dance, dispelling gloom.
Sizes shift with a gentle sway,
A perfect fit for every play.
With colors bright and styles anew,
Our buttons shine, just for you! 🌼✨
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
PR
PR Checklist
Please check if your PR fulfills the following requirements:
PR Type
What kind of change does this PR introduce?
What is the current behavior?
Issue Number: N/A
What is the new behavior?
Does this PR introduce a breaking change?
Other information
Summary by CodeRabbit
New Features
Bug Fixes