-
Notifications
You must be signed in to change notification settings - Fork 14.6k
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
Table: fix the bug setting default-sort to Table triggers sort-change… #14625
Conversation
Deploy preview for element ready! Built with commit d0164d5 |
Any news on this please? |
@PeanutWatson 修改一下。另外这个属性 |
好滴,多谢指教,我修改一下。 |
…directly Last commit failed by trailing spaces, fix it.
…directly (ElemeFE#13750) Change the code according the reviewer advises.
@ziyoung 已做了修改,将isDefault改为init并提至table-header中的mounted函数中传到名为sort的 mutation中,不知这样可以吗?如果可以,是否可以把init提到table-header的 props中的defaultSort的default属性中? |
@PeanutWatson 帮你调整了一下的部分代码。核心逻辑没变。 |
|
@PeanutWatson 如果还有 bug 的话,提供一个链接吧。 |
@ziyoung 2.11又重现了 |
在用table组件的时候sort-change 会触发两次http的请求,有遇到过的吗,如何解决 |
Fix the bug when setting default-sort prop to Table, it will trigger sort-change function directly.
To Closes #13750