Skip to content

New table #380

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 84 commits into from
Oct 30, 2019
Merged

New table #380

merged 84 commits into from
Oct 30, 2019

Conversation

zombieJ
Copy link
Member

@zombieJ zombieJ commented Oct 28, 2019 •
edited
Loading

Valuable Preview

zombieJ added 30 commits October 16, 2019 17:00
@zombieJ
basic init
146cd09
@zombieJ
Cell render
73d821d
@zombieJ
add flatten logic
1162c70
@zombieJ
refactor of memo component
07e0c50
@zombieJ
support merge title
ea0cb4a
@zombieJ
add simple example
bed5d00
@zombieJ
add fixHedaer
0bff0fb
@zombieJ
sticky header
084e554
@zombieJ
supoor ref
743c7be
@zombieJ
support scroll
427e0fc
@zombieJ
fix right
1663959
@zombieJ
sticky width
ad92517
@zombieJ
fixed support merged column
8798769
@zombieJ
box-shadow of fixed
fb03cbd
@zombieJ
fix scroll crash
7fddbac
@zombieJ
update ts
77a13dd
@zombieJ
add basic expanded
4aaf679
@zombieJ
expandable support style update
24c2e7b
@zombieJ
add addtional column of scroll bar
70d17d3
@zombieJ
make fix more beauti
b106696
@zombieJ
row click expand
a78490e
@zombieJ
support rowExpandable
babaf12
@zombieJ
fix click support row expandable
e50b17a
@zombieJ
limit size
0d0d07a
@zombieJ
clean up
f5fb6ab
@zombieJ
add word break
f8f7699
@zombieJ
add colWidth to force use table width
5937dad
@zombieJ
re-onScroll if size changed
648eb1d
@zombieJ
update demo
6ef8538
@zombieJ
fix scroll class missing
86622ae
@lgtm-com
Copy link
Contributor

lgtm-com bot commented Oct 29, 2019

This pull request fixes 1 alert when merging a214e9c into 04de08d - view on LGTM.com

fixed alerts:

  • 1 for Missing space in string concatenation

@lgtm-com
Copy link
Contributor

lgtm-com bot commented Oct 29, 2019

This pull request fixes 1 alert when merging bd965c7 into 04de08d - view on LGTM.com

fixed alerts:

  • 1 for Missing space in string concatenation

@lgtm-com
Copy link
Contributor

lgtm-com bot commented Oct 29, 2019

This pull request fixes 1 alert when merging c63b39c into 04de08d - view on LGTM.com

fixed alerts:

  • 1 for Missing space in string concatenation

@lgtm-com
Copy link
Contributor

lgtm-com bot commented Oct 30, 2019

This pull request fixes 1 alert when merging 5f8c8ae into 04de08d - view on LGTM.com

fixed alerts:

  • 1 for Missing space in string concatenation

@lgtm-com
Copy link
Contributor

lgtm-com bot commented Oct 30, 2019

This pull request fixes 1 alert when merging df5c365 into 04de08d - view on LGTM.com

fixed alerts:

  • 1 for Missing space in string concatenation

@lgtm-com
Copy link
Contributor

lgtm-com bot commented Oct 30, 2019

This pull request fixes 1 alert when merging 9c43b70 into 04de08d - view on LGTM.com

fixed alerts:

  • 1 for Missing space in string concatenation

@lgtm-com
Copy link
Contributor

lgtm-com bot commented Oct 30, 2019

This pull request fixes 1 alert when merging aedcdcd into 04de08d - view on LGTM.com

fixed alerts:

  • 1 for Missing space in string concatenation

@zombieJ zombieJ marked this pull request as ready for review October 30, 2019 07:05
@lgtm-com
Copy link
Contributor

lgtm-com bot commented Oct 30, 2019

This pull request fixes 1 alert when merging 06e2d73 into 04de08d - view on LGTM.com

fixed alerts:

  • 1 for Missing space in string concatenation

@zombieJ zombieJ changed the title [WIP] New table New table Oct 30, 2019
@zombieJ zombieJ merged commit 55e4bfb into master Oct 30, 2019
@delete-merged-branch delete-merged-branch bot deleted the new-table branch October 30, 2019 07:21
@afc163 afc163 mentioned this pull request Nov 4, 2019
Closed
1 task
crazyair
crazyair reviewed Sep 1, 2025
View reviewed changes
src/ColGroup.tsx
export interface ColGroupProps<RecordType> {
colWidths: (number | string)[];
columns?: ColumnType<RecordType>[];
columCount?: number;
Copy link
Contributor

Choose a reason for hiding this comment

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

columnCount

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
1 more reviewer

@crazyair crazyair crazyair left review comments

Reviewers whose approvals may not affect merge requirements
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@zombieJ @crazyair