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

[docs][Joy] Add documentation for Joy Grid component #35374

Merged
merged 35 commits into from Dec 26, 2022

Conversation

hbjORbj
Copy link
Member

@hbjORbj hbjORbj commented Dec 6, 2022

@hbjORbj hbjORbj self-assigned this Dec 6, 2022
@hbjORbj hbjORbj added docs Improvements or additions to the documentation component: Grid The React component. package: joy-ui Specific to @mui/joy labels Dec 6, 2022
@mui-bot
Copy link

mui-bot commented Dec 6, 2022

Messages
📖 Netlify deploy preview: https://deploy-preview-35374--material-ui.netlify.app/

No bundle size changes

Generated by 🚫 dangerJS against b47a19c

docs/data/joy/pages.ts Outdated Show resolved Hide resolved
Copy link
Member

@siriwatknp siriwatknp left a comment

Choose a reason for hiding this comment

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

  1. Please follow the Table of contents structure from the page that @samuelsycamore has done, .e.g. the AspectRatio page.
  2. The Interactive demo does not look good.
    Screen Shot 2565-12-19 at 11 44 12
  3. Let's remove the "Complex Grid" demo. I don't think it is a good example.
    https://deploy-preview-35374--material-ui.netlify.app/material-ui/react-grid2/#customization
  4. Add these examples similar to the Material UI Grid v2 to show the common examples.

@hbjORbj
Copy link
Member Author

hbjORbj commented Dec 19, 2022

  1. Done but I think we still need @samuelsycamore 's help to polish it further.
  2. Done.
  3. Done.

@hbjORbj
Copy link
Member Author

hbjORbj commented Dec 21, 2022

  1. Add these examples similar to the Material UI Grid v2 to show the common examples.

I added a common example that shows how to center children of grid item and how to set borders on grid. I am open for suggestions.

Copy link
Member

@siriwatknp siriwatknp left a comment

Choose a reason for hiding this comment

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

Please check my comment before merging.

@github-actions github-actions bot added the PR: out-of-date The pull request has merge conflicts and can't be merged label Dec 26, 2022
@github-actions github-actions bot removed the PR: out-of-date The pull request has merge conflicts and can't be merged label Dec 26, 2022
@hbjORbj hbjORbj merged commit a2436fc into mui:master Dec 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: Grid The React component. docs Improvements or additions to the documentation package: joy-ui Specific to @mui/joy
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants