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): [carousel] two length transform optimize #12174

Merged
merged 12 commits into from
Oct 11, 2023

Conversation

MrWeilian
Copy link
Contributor

fix: #12139

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 23, 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 23, 2023

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

@MrWeilian
Copy link
Contributor Author

playground

@makedopamine
Copy link
Contributor

Hi, I notice there is another situation that needs to be dealt with. Specifically, the addtional element appear in card mode.

@MrWeilian
Copy link
Contributor Author

o~ thank you very much, it is resolved now

@MrWeilian
Copy link
Contributor Author

@jw-foss hi, May I request review from you? And I don't know if this issue can be resolved in this way

@xtthaop
Copy link

xtthaop commented Aug 3, 2023

I think this problem shuold be solved, but they seem too busy.

@MrWeilian
Copy link
Contributor Author

@tolking @btea @chenxch 大佬们有空cc

@HeftyKoo HeftyKoo merged commit d88cb75 into element-plus:dev Oct 11, 2023
10 checks passed
@element-bot element-bot mentioned this pull request Oct 13, 2023
3 tasks
cc-hearts pushed a commit to cc-hearts/element-plus that referenced this pull request Oct 15, 2023
…us#12174)

* fix(components): [carousel] two length transform optimize

* fix(components): [carousel] default init function slot

* fix(components): [carousel] cancel optimize when loop is false

* fix(components): [carousel] filter card type

* fix: test

* style: code lint

* fix: cardtype

* fix: remove test code

* fix: act index

* feat: use flattedchildren

* fix: code lint

* fix: rename and add comment
consultation-applio pushed a commit to consultation-applio/element-plus that referenced this pull request Nov 10, 2023
…us#12174)

* fix(components): [carousel] two length transform optimize

* fix(components): [carousel] default init function slot

* fix(components): [carousel] cancel optimize when loop is false

* fix(components): [carousel] filter card type

* fix: test

* style: code lint

* fix: cardtype

* fix: remove test code

* fix: act index

* feat: use flattedchildren

* fix: code lint

* fix: rename and add comment
@guda-art
Copy link

guda-art commented Jun 1, 2024

抱歉, 我将使用中文, 现在看来这个改动有很大的副作用 参见#15086
原element(vue2.x) 仿佛从未解决过这个问题, 应该有更加周全的考虑

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