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] backspace delete disabled option #11995

Merged
merged 5 commits into from
Aug 13, 2023

Conversation

MrWeilian
Copy link
Contributor

closed: #11991

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 Mar 12, 2023

@github-actions
Copy link

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 Mar 12, 2023

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

@MrWeilian
Copy link
Contributor Author

playground

@ryuhangyeong
Copy link
Member

LGTM 😃

@btea
Copy link
Collaborator

btea commented Aug 10, 2023

@MrWeilian Could you resolve the code conflict?

@MrWeilian
Copy link
Contributor Author

@MrWeilian Could you resolve the code conflict?

done. but I found that the test is error with the avatar.test.tsx ...

@btea
Copy link
Collaborator

btea commented Aug 11, 2023

@MrWeilian Don't worry, the test case is sometimes unstable, it will be fine after re-running.

@btea btea requested a review from tolking August 13, 2023 10:16
@btea btea merged commit 067028b into element-plus:dev Aug 13, 2023
10 checks passed
@element-bot element-bot mentioned this pull request Aug 25, 2023
3 tasks
laizuan pushed a commit to laizuan/element-plus that referenced this pull request Sep 5, 2023
…lus#11995)

* fix(components): [select] backspace delete disabled option

* fix(components): [select] findLastIndex

* fix(components): [select] simple polyfill findLastIndex in test file

* fix(components): [select] add test for backspace

* chore: lint
@wforguo
Copy link

wforguo commented Sep 12, 2023

七月份提的pr,没给我合,合了一个八月份的,妙啊

#13690

@MrWeilian
Copy link
Contributor Author

MrWeilian commented Sep 12, 2023

建议提 pr 的时候可以关联一下issue,并看看有没相关的解决。。。 #11995 是三月份提的。。。

consultation-applio pushed a commit to consultation-applio/element-plus that referenced this pull request Nov 10, 2023
…lus#11995)

* fix(components): [select] backspace delete disabled option

* fix(components): [select] findLastIndex

* fix(components): [select] simple polyfill findLastIndex in test file

* fix(components): [select] add test for backspace

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

Successfully merging this pull request may close these issues.

[Component] [select] el-select mutiple filterable could use backspace key to delete disabled options
5 participants