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

[DataGrid] Remove old string class constants #2788

Merged
merged 5 commits into from
Oct 12, 2021

Conversation

flaviendelangle
Copy link
Member

@flaviendelangle flaviendelangle commented Oct 6, 2021

Part of #2227

Breaking changes

-const columnHeaderClass = GRID_COLUMN_HEADER_CSS_CLASS
+const columnHeaderClass = gridClasses.columnHeader

-const rowClass = GRID_ROW_CSS_CLASS
+const rowClass = gridClasses.row

-const cellClass = GRID_CELL_CSS_CLASS
+const cellClass = gridClasses.cell

-const columnSeparatorClass = GRID_COLUMN_HEADER_SEPARATOR_RESIZABLE_CSS_CLASS
+const columnSeparatorClass = gridClasses['columnSeparator--resizable']

-const columnHeaderTitleClass = GRID_COLUMN_HEADER_TITLE_CSS_CLASS
+const columnHeaderTitleClass = gridClasses.columnHeaderTitle

-const columnHeaderDropZoneClass = GRID_COLUMN_HEADER_DROP_ZONE_CSS_CLASS
+const columnHeaderDropZoneClass = gridClasses.columnHeaderDropZone

-const columnHeaderDraggingClass = GRID_COLUMN_HEADER_DRAGGING_CSS_CLASS
+const columnHeaderDraggingClass = gridClasses["columnHeader--dragging"]

@flaviendelangle flaviendelangle self-assigned this Oct 6, 2021
@flaviendelangle flaviendelangle added this to the v5 stable version milestone Oct 6, 2021
@flaviendelangle flaviendelangle added breaking change component: data grid This is the name of the generic UI component, not the React module! labels Oct 6, 2021
@flaviendelangle flaviendelangle merged commit 8f9fcc5 into mui:next Oct 12, 2021
@flaviendelangle flaviendelangle deleted the classes branch October 12, 2021 14:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking change component: data grid This is the name of the generic UI component, not the React module!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants