Closed Youyou-smiles closed 2 weeks ago
The changes in this pull request focus on the index.less
file for the button component, specifically enhancing the layout and alignment of "only icon" buttons. Key modifications include the introduction of inline-flex
display properties, centering adjustments for icon alignment, and the removal of padding for the .is-only-icon
class. Additionally, the handling of button sizes is updated to ensure responsive icon sizes, contributing to a more visually consistent presentation of icon-only buttons.
File Path | Change Summary |
---|---|
packages/theme/src/button/index.less | - Added display: inline-flex , align-items: center , and justify-content: center to .is-only-icon class. - Set padding to 0 for .is-only-icon class. - Updated icon sizing management with .size-only-icon-mixin and .size-only-icon-ontext-iconsize-mixin . |
index.less
to add a new CSS variable for the .is-ghost
class, affecting button styling..is-only-icon
class, directly relating to icon-only button layout..is-only-icon
state, enhancing functionality for icon-only buttons..is-icon
class to set its display to inline-flex
and align items to the center, relevant to icon button layout improvements.enhancement
🐰 In the meadow, buttons shine,
Icons centered, oh so fine.
Padding gone, they stand so tall,
A joyful change, we love them all!
Hopping forth with style and grace,
Icon buttons find their place! 🌼
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