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

fix(components): [select] support custom height (#10840) #10847

Merged
merged 2 commits into from
Feb 27, 2023

Conversation

MrWeilian
Copy link
Contributor

@MrWeilian MrWeilian commented Dec 1, 2022

Fix: [Component] [select] el-select的disabled属性切换时,如果自定义了line-height,里面input框的高度计算有问题
Please make sure these boxes are checked before submitting your PR, thank you!

  • Make sure you follow contributing guide English | (中文 | Español | Français).
  • Make sure you are merging your commits to dev branch.
  • Add some descriptions and refer to relative issues for your PR.

@pull-request-triage
Copy link

👋 @MrWeilian, thank you for contributing element-plus.

  • You can comment with /label Components:[component_name] to add a label for which component you are working on.
  • You may join our Discord for staying tuned.

@github-actions
Copy link

github-actions bot commented Dec 1, 2022

@github-actions
Copy link

github-actions bot commented Dec 1, 2022

Hello @MrWeilian, thank you for contributing to element-plus, please see our guideline to see how to make contribution

@github-actions
Copy link

github-actions bot commented Dec 1, 2022

🧪 Playground Preview: https://element-plus.run/?pr=10847
Please comment the example via this playground if needed.

@MrWeilian
Copy link
Contributor Author

MrWeilian commented Dec 1, 2022

playground

@u373693041
Copy link

Could this change be reviewed as soon as possible please? This bug has a great impact on me, thank you

@MrWeilian
Copy link
Contributor Author

Could this change be reviewed as soon as possible please? This bug has a great impact on me, thank you

@emojiiii @chenxch 大佬们

@chenxch
Copy link
Member

chenxch commented Feb 11, 2023

demo
There is also an initialization problem. I remember that there is a related pr, and I will look for it.

@chenxch
Copy link
Member

chenxch commented Feb 11, 2023

#11374
Of course this should be another issue.

Copy link
Member

@chenxch chenxch left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good job.

@MrWeilian
Copy link
Contributor Author

good job.

猴 我更新下分支

@MrWeilian
Copy link
Contributor Author

😂 大佬们 这个pr能合并吗 又解决一次冲突啦🙈

@chenxch chenxch merged commit 2f5629a into element-plus:dev Feb 27, 2023
@element-bot element-bot mentioned this pull request Mar 3, 2023
3 tasks
loosheng pushed a commit to loosheng/element-plus that referenced this pull request Mar 26, 2023
…lement-plus#10847)

* fix(components): [select] support custom height (element-plus#10840)

fix(components): [select] support custom height (element-plus#10840)

* fix(components): [select] support custom height (element-plus#10840)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants