Grid2 and Fragment: missing padding on Grid item

Open
#37,062 4 comments 1 reaction 1 assignee View on GitHub

@siriwatknp is already working on this.

Since Apr 27, 2023.

Assessment

This issue has not been assessed yet.

Description

component: Grid has workaround scope: system type: enhancement
Duplicates
  • I have searched the existing issues
Latest version
  • I have tested the latest version
Steps to reproduce 🕹

Link to live example:
https://codesandbox.io/s/festive-ully-7fjqbf

Current behavior 😯

In the provided example the grid item "L2 (Fragment) - Node" doesn't get the padding.

Expected behavior 🤔

In the provided example the grid item "L2 (Fragment) - Node" should get the padding according to the container.

Context 🔦

I have a complex structure with nested Grid2 containers, the new implementation (from v5.12.1) breaks the layout of my components.

Your environment 🌎
npx @mui/envinfo 
  System:
    OS: macOS 13.3.1
  Binaries:
    Node: 20.0.0 - /usr/local/bin/node
    Yarn: 3.4.1 - /opt/homebrew/bin/yarn
    npm: 9.6.4 - /usr/local/bin/npm
  Browsers:
    Chrome: 112.0.5615.137
    Edge: Not Found
    Firefox: 112.0.2
    Safari: 16.4
  npmPackages:
    @emotion/react: ^11.10.6 => 11.10.6 
    @emotion/styled: ^11.10.6 => 11.10.6 
    @mui/base: ^5.0.0-alpha.127 => 5.0.0-alpha.127 
    @mui/core-downloads-tracker:  5.12.2 
    @mui/icons-material: ^5.11.16 => 5.11.16 
    @mui/lab: ^5.0.0-alpha.128 => 5.0.0-alpha.128 
    @mui/material: ^5.12.2 => 5.12.2 
    @mui/private-theming:  5.12.0 
    @mui/styled-engine:  5.12.0 
    @mui/system: ^5.12.1 => 5.12.1 
    @mui/types:  7.2.4 
    @mui/utils:  5.11.13 
    @mui/x-data-grid:  6.2.1 
    @mui/x-data-grid-premium: ^6.2.1 => 6.2.1 
    @mui/x-data-grid-pro:  6.2.1 
    @mui/x-date-pickers:  6.2.1 
    @mui/x-date-pickers-pro: ^6.2.1 => 6.2.1 
    @mui/x-license-pro:  6.0.4 
    @types/react:  18.0.14 
    react: ^18.2.0 => 18.2.0 
    react-dom: ^18.2.0 => 18.2.0 
    typescript:  4.7.4

I'm using Google Chrome

Dominant language
JavaScript
Stars
99.1k
Forks
32.5k
Avg merge
2d 17h
Merged PRs (30d)
106

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from mui/material-ui

All issues in mui/material-ui

Similar issues

More JavaScript issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.