Skip to content
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

style: fix input width overflow in Transfer&ColorPicker #2664

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

T-Roc
Copy link
Contributor

@T-Roc T-Roc commented Apr 28, 2024

Types of changes

  • Bug fix

Background and context

问题:
分析 issues #2627,并排查所有组件发现:

  • Transfer、ColorPicker 都会被 ConfigProvider 中设置了 Input 的宽度影响样式
  • 其他内部依赖 Input 组件的不会被影响,包括:Cascader、Slider、VerificationCode、AutoComplete、Select、InputTag、InputNumber、Mentions
image

Solution

  • 修改 Transfer、ColorPicker 中的宽度限制

How is the change tested?

Changelog

Component Changelog(CN) Changelog(EN) Related issues
Transfer 修复 Transfer 组件因为 ConfigProvider 组件中设置了 Input 的宽度造成样式错乱问题 Fix the styling issue in the Transfer component caused by the width setting of Input in the ConfigProvider component #2627
ColorPicker 修复 ColorPicker 组件因为 ConfigProvider 组件中设置了 Input 的宽度造成样式错乱问题 Fix the styling issue in the ColorPicker component caused by the width setting of Input in the ConfigProvider component

Checklist:

  • Test suite passes (npm run test)
  • Provide changelog for relevant changes (e.g. bug fixes and new features) if applicable.
  • Changes are submitted to the appropriate branch (e.g. features should be submitted to feature branch and others should be submitted to main branch)

Other information

Copy link

codesandbox bot commented Apr 28, 2024

Review or Edit in CodeSandbox

Open the branch in Web EditorVS CodeInsiders

Open Preview

Copy link

Prepare preview

Copy link

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant