Skip to content

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

Merged
apedroferreira merged 1 commit intomui: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 intomui:masterfrom
apedroferreira:fix-material-ui-drawer-docs

Conversation

@apedroferreira
Copy link
Collaborator

@apedroferreira apedroferreira commented Mar 26, 2025

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
Copy link

mui-bot commented Mar 26, 2025

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
Copy link
Member

Janpot commented Mar 26, 2025

Interesting that no eslint rule exists to catch that unused ref

@apedroferreira
Copy link
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
21 checks passed
@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