Skip to content

[datagrid] Pass type="button" to GridLongTextCellCornerButton to avoid triggering form submission#22383

Closed
flvinny wants to merge 1 commit into
mui:masterfrom
flvinny:longtext-corner-button-form-submit
Closed

[datagrid] Pass type="button" to GridLongTextCellCornerButton to avoid triggering form submission#22383
flvinny wants to merge 1 commit into
mui:masterfrom
flvinny:longtext-corner-button-form-submit

Conversation

@flvinny

@flvinny flvinny commented May 8, 2026

Copy link
Copy Markdown

Fixes #22382

@code-infra-dashboard

Copy link
Copy Markdown

Deploy preview

https://deploy-preview-22383--material-ui-x.netlify.app/

Bundle size

Bundle Parsed size Gzip size
@mui/x-data-grid 🔺+28B(+0.01%) 🔺+9B(+0.01%)
@mui/x-data-grid-pro 🔺+28B(+0.01%) 🔺+8B(+0.01%)
@mui/x-data-grid-premium 🔺+28B(0.00%) 🔺+7B(0.00%)
@mui/x-charts 0B(0.00%) 0B(0.00%)
@mui/x-charts-pro 0B(0.00%) 0B(0.00%)
@mui/x-charts-premium 0B(0.00%) 0B(0.00%)
@mui/x-date-pickers 0B(0.00%) 0B(0.00%)
@mui/x-date-pickers-pro 0B(0.00%) 0B(0.00%)
@mui/x-tree-view 0B(0.00%) 0B(0.00%)
@mui/x-tree-view-pro 0B(0.00%) 0B(0.00%)
@mui/x-license 0B(0.00%) 0B(0.00%)

Details of bundle changes


Check out the code infra dashboard for more information about this PR.

@zannager zannager added the scope: data grid Changes related to the data grid. label May 8, 2026

@mj12albert mj12albert left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@flvinny Could you add appropriate tests as well?

@mj12albert mj12albert added the PR: needs test The pull request needs tests. label May 13, 2026
@mj12albert

Copy link
Copy Markdown
Member

#22450

@mj12albert mj12albert closed this May 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PR: needs test The pull request needs tests. scope: data grid Changes related to the data grid.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[data grid] Clicking a longText column's expand button triggers a form submission

3 participants