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
feat(react-grid): implement column chooser #366
Conversation
columns | Array<[Column](grid.md#column)> | | Specifies for which row object fields columns are created. | ||
hiddenColumns | Array<string> | [] | An array containing the names of the columns to be hidden. | ||
onHiddenColumnsChange | (nextHiddenColumns: Array<string>) => void | | Handles column visibility change. | ||
contentTemplate | (args: [ColumnChooserRootArgs](#column-chooser-root-args)) => ReactElement | | A template that renders the column chooser markup. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
contentTemplate with ColumnChooserRootArgs... It's quite strange
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also seems that we should extract container and item templates for API consistency
@@ -0,0 +1,30 @@ | |||
# HiddenTableColumns Plugin Reference |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Outdated?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Don't forget to update landing page
@@ -0,0 +1,38 @@ | |||
# Grid Reference |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why 'Grid'?
Field | Type | Description | ||
------|------|------------ | ||
columnChooserItems | Array<[ColumnChooserItem](#column-chooser-item)> | An array of column chooser items. | ||
onColumnToggle | (columnName: string) => void | Handles column visibility change. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Or onToggleColumn?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Discussed this with our technical writer. The onColumnToggle
is correct.
d8bb399
to
f5b38f0
Compare
}; | ||
``` | ||
|
||
At this point, the Grid and the ColumnChooser a synchronized. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"At this point, the Grid and the ColumnChooser a synchronized."?
No description provided.