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

Optimize table selection #3757

Merged
merged 2 commits into from
Nov 11, 2016
Merged

Optimize table selection #3757

merged 2 commits into from
Nov 11, 2016

Conversation

yesmeck
Copy link
Member

@yesmeck yesmeck commented Nov 7, 2016

这个 PR 主要做了跟 react-component/table#95 类似的事情,把 selectedRowKeys 这个状态提出来,当有选择变化的时候只需要重新渲染对应的 cell。

另外 columns 还是需要 cache 的,具体想法见这里 #2884 (comment)

@mention-bot
Copy link

@yesmeck, thanks for your PR! By analyzing the history of the files in this pull request, we identified @benjycui, @ddcat1115 and @afc163 to be potential reviewers.

@yesmeck
Copy link
Member Author

yesmeck commented Nov 8, 2016

这个 CI 很奇怪啊,一个是好的,一个是坏的。

@benjycui
Copy link
Contributor

benjycui commented Nov 8, 2016

Rebase.

@yesmeck yesmeck changed the title [WIP]Optimize table selection Optimize table selection Nov 8, 2016
@yesmeck yesmeck changed the title Optimize table selection [WIP]Optimize table selection Nov 8, 2016
@yesmeck yesmeck changed the title [WIP]Optimize table selection Optimize table selection Nov 8, 2016
@yesmeck yesmeck changed the title Optimize table selection [WIP]Optimize table selection Nov 8, 2016
@yesmeck yesmeck changed the title [WIP]Optimize table selection Optimize table selection Nov 9, 2016
@afc163 afc163 merged commit 0e6ac6b into master Nov 11, 2016
@afc163 afc163 deleted the optimize-table branch November 11, 2016 07:26
jljsj33 pushed a commit that referenced this pull request Nov 11, 2016
* Extract SelectionRadio, SelectionCheckbox, SelectionCheckboxAll

* Add some tests
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

4 participants