feat: remove medium form width and modify tag styles#586
feat: remove medium form width and modify tag styles#586yangxiaolang merged 6 commits intomasterfrom
Conversation
🦋 Changeset detectedLatest commit: d2b9cfa The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
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 |
WalkthroughThe updates adjust default styling and documentation for tag and form components. Tag components now have non-rounded corners by default, with related documentation and stories updated accordingly. Multi-select tag appearance is also changed to non-rounded and borderless. The medium form item width is deprecated, and its CSS variable is updated to a new width. Changes
Poem
📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (5)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (2)
src/form/form-item/form-item.component.scss (1)
88-96: Avoid hard-coding the key list – iterate the map itselfManually listing the keys (
large small) duplicates source-of-truth in the loop and will drift if new sizes are ever introduced.-@each $key in large small { - $map: map-get($form-item-width, $key); +@each $key, $map in $form-item-width { &--#{$key} { .aui-form-item__control { flex: 1; max-width: map-get($map, width); } } }src/select/multi-select/multi-select.component.html (1)
44-45: Explicitfalsebindings may be unnecessary noiseIf the
<aui-tag>inputs default tofalse, the explicit[round]="false"/[border]="false"can be dropped for a cleaner template.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (5)
src/form/form-item/form-item.component.scss(1 hunks)src/form/form.types.ts(0 hunks)src/select/multi-select/multi-select.component.html(1 hunks)src/tag/tag.component.scss(1 hunks)src/theme/_theme-preset.scss(0 hunks)
💤 Files with no reviewable changes (2)
- src/theme/_theme-preset.scss
- src/form/form.types.ts
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (5)
- GitHub Check: Redirect rules - aui
- GitHub Check: Header rules - aui
- GitHub Check: Pages changed - aui
- GitHub Check: Release Beta
- GitHub Check: Unit Test
🔇 Additional comments (1)
src/form/form-item/form-item.component.scss (1)
4-11: Droppedmediumreferences verified
I’ve searched theform-itemcomponent (SCSS/HTML/TS) and the wider codebase for any remainingmediumsize usages—CSS vars, template bindings, enums, or BEM modifiers—and found none. Themediumentry can safely stay removed without dangling references.
|
PR 标题写规范点儿吧 |
style: update tag info bg style: tag not default round chore: modify tag story
chore: compliant medium with large size
feat: review change feat: review change
1906183 to
fde27e2
Compare
Co-authored-by: JounQin <admin@1stg.me>
Summary by CodeRabbit
Style
Documentation
New Features
Refactor