Skip to content

[docs] Fix blank space issue in Material UI Drawer docs - #4805

Merged
apedroferreira merged 1 commit into
mui:masterfrom
apedroferreira:fix-material-ui-drawer-docs
Mar 26, 2025
Merged

[docs] Fix blank space issue in Material UI Drawer docs#4805
apedroferreira merged 1 commit into
mui:masterfrom
apedroferreira:fix-material-ui-drawer-docs

Conversation

@apedroferreira

@apedroferreira apedroferreira commented Mar 26, 2025

Copy link
Copy Markdown
Collaborator

I've tested it and this should fix the big blank space at the bottom of https://master--material-ui.netlify.app/material-ui/react-drawer/#dashboard-layout

@apedroferreira apedroferreira added the docs Improvements or additions to the documentation. label Mar 26, 2025
@apedroferreira apedroferreira self-assigned this Mar 26, 2025
@mui-bot

mui-bot commented Mar 26, 2025

Copy link
Copy Markdown

Netlify deploy preview

https://deploy-preview-4805--mui-toolpad-docs.netlify.app/

Generated by 🚫 dangerJS against 88aaa10

@apedroferreira
apedroferreira marked this pull request as ready for review March 26, 2025 16:55
@apedroferreira
apedroferreira requested a review from a team March 26, 2025 16:55
@apedroferreira apedroferreira changed the title [docs] Attempt to solve blank space issue in Material UI Drawer docs [docs] Fix blank space issue in Material UI Drawer docs Mar 26, 2025
@Janpot

Janpot commented Mar 26, 2025

Copy link
Copy Markdown
Member

Interesting that no eslint rule exists to catch that unused ref

@apedroferreira

Copy link
Copy Markdown
Collaborator Author

Interesting that no eslint rule exists to catch that unused ref

Well it was being used for the container, but once I removed it there was no eslint warning even though the ref wasn't doing anything anymore, I guess there might be a React rule for that.

@apedroferreira
apedroferreira merged commit 4c4a232 into mui:master Mar 26, 2025
@apedroferreira
apedroferreira deleted the fix-material-ui-drawer-docs branch March 26, 2025 18:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs Improvements or additions to the documentation.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants