Closed zzcr closed 2 days ago
The changes primarily involve the removal of border styles from various CSS classes across multiple Vue component files. This alteration affects the visual styling of components, specifically those using the .split-v-model
and .demo-split
classes, by eliminating previously defined borders. Additionally, modifications were made to test files to change assertion methods from toBeCloseTo
to toEqual
, adjusting the precision of comparisons in tests.
File(s) | Change Summary |
---|---|
examples/sites/demos/pc/app/split/*.vue |
Removed border: 1px solid #d9d9d9; from .split-v-model and .demo-split classes in various files, altering their visual styling. |
examples/sites/demos/pc/app/split/*.spec.ts |
Updated assertions from toBeCloseTo to toEqual with Math.ceil() for stricter comparison in test cases. |
packages/theme/src/split/index.less |
Replaced .component-css-vars-split() with .inject-Split-vars() , updated styles and dimensions using CSS variables. |
packages/theme/src/split/vars.less |
Removed .component-css-vars-split() and introduced .inject-Split-vars() with new CSS variable definitions. |
vars.less
, indirectly relating to variable usage in the main PR.refactoring
🐇 In a meadow bright and fair,
Borders gone, no more to wear.
Components dance with styles anew,
A cleaner look, a fresher view.
Tests now precise, no room for doubt,
Hooray for changes, let’s hop about! 🐇
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
Documentation