-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
[DataGrid] Add api pages for data-grid and x-grid #289
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.
Great start!
We would need two new pages to have them rendered:
- /docs/pages/api-docs/data-grid.js
- /docs/pages/api-docs/x-grid.js
Also, the convention is to add a hyphen when we use an uppercase so:
- /docs/pages/api-docs/data-grid.md
- /docs/pages/api-docs/x-grid.md
See the link generated automatically:
docs/pages/api-docs/xgrid.md
Outdated
|
||
| Rule name | Global class | Description | | ||
|:-----|:-------------|:------------| | ||
| <span class="prop-name">root</span> | <span class="prop-name">.MuiDataGrid-root</span> | Styles applied to the root element. |
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.
Does the classes
prop work? It seems it doesn't.
| <span class="prop-name">root</span> | <span class="prop-name">.MuiDataGrid-root</span> | Styles applied to the root element. | |
| <span class="prop-name"></span> | <span class="prop-name">.MuiDataGrid-root</span> | Styles applied to the root element. |
docs/pages/api-docs/xgrid.md
Outdated
| Rule name | Global class | Description | | ||
|:-----|:-------------|:------------| | ||
| <span class="prop-name">root</span> | <span class="prop-name">.MuiDataGrid-root</span> | Styles applied to the root element. | ||
| <span class="prop-name">root</span> | <span class="prop-name">.MuiDataGrid-mainGridContainer</span> | Styles applied to the main container element. |
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.
root
is not implemented here and for all the other keys.
| <span class="prop-name">root</span> | <span class="prop-name">.MuiDataGrid-mainGridContainer</span> | Styles applied to the main container element. | |
| <span class="prop-name"></span> | <span class="prop-name">.MuiDataGrid-mainGridContainer</span> | Styles applied to the main container element. |
Actually, we could implement them with the context.
Co-authored-by: Olivier Tassinari <olivier.tassinari@gmail.com>
I have taken note to fix the link underline in the header in the next batch of small changes for the docs |
No description provided.