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

refactor(react-grid): replace render functions with components in TableHeaderRow #508

Merged
merged 14 commits into from Dec 5, 2017
Merged

refactor(react-grid): replace render functions with components in TableHeaderRow #508

merged 14 commits into from Dec 5, 2017

Conversation

kvet
Copy link
Contributor

@kvet kvet commented Nov 24, 2017

BREAKING CHANGES:

The TableHeaderRow's headerCellTemplate, and headerRowTemplate properties have been replaced with getCellComponent, and rowComponent, which accept components instead of render functions. Find more details here: #496

Properties passed to getCellComponent have the same names as arguments passed to the headerCellTemplate function with the following exceptions: the onSort, onGroup, onWidthChange and onDraftWidthChange properties are used instead of the changeSortingDirection, groupByColumn, changeColumnWidth and changeDraftColumnWidth arguments respectively.

@kvet kvet changed the title refactor(react-grid): replace render functions with components in TableHeaderRow [wip]refactor(react-grid): replace render functions with components in TableHeaderRow Nov 24, 2017
@kvet kvet changed the title [wip]refactor(react-grid): replace render functions with components in TableHeaderRow refactor(react-grid): replace render functions with components in TableHeaderRow Nov 27, 2017
@kvet kvet requested a review from viterobk November 27, 2017 09:41
@kvet kvet merged commit 255e729 into DevExpress:master Dec 5, 2017
@kvet kvet deleted the tableheaderrow-components branch December 5, 2017 14:13
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

5 participants