-
Notifications
You must be signed in to change notification settings - Fork 25
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
sample(grid): Add export visualiuzation sample #2233
Conversation
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.
- Remove hierarchical grid sample since we do not officially support exporting for hierarchical grid.
- Add some margin in the sample so the grid border shadow is not cut off in docfx.
- Since the CSV export is faster, the exporting indicator does not appear when exporting to CSV. We should either disable the CSV export or additionally increase the amount of data.
src/app/grid/grid-export-visualization/grid-export-visualization.component.html
Outdated
Show resolved
Hide resolved
src/app/grid/grid-export-visualization/grid-export-visualization.component.html
Outdated
Show resolved
Hide resolved
src/app/tree-grid/tree-grid-export-visualization/tree-grid-export-visualization.component.html
Outdated
Show resolved
Hide resolved
src/app/grid/grid-export-visualization/grid-export-visualization.component.scss
Outdated
Show resolved
Hide resolved
src/app/grid/grid-export-visualization/grid-export-visualization.component.html
Show resolved
Hide resolved
src/app/tree-grid/tree-grid-export-visualization/tree-grid-export-visualization.component.html
Outdated
Show resolved
Hide resolved
I verified the PR but you have lint errors to fix. |
|
Oops, my bad. I commented on the wrong PR. |
Closes #2212