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-infra] Share code for section title #42236

Merged
merged 1 commit into from
May 15, 2024

Conversation

alexfauquette
Copy link
Member

Export a component to render a section title.

Pages are still responsible for the definition of the translation since they have different approaches. But at least the classes and the DOM structure will be shared.

Not sure about the file position. I hesitated to put this component inside a folder containing small UI components, but did not find a name for it

@alexfauquette alexfauquette added the scope: docs-infra Specific to the docs-infra product label May 14, 2024
@mui-bot
Copy link

mui-bot commented May 14, 2024

Netlify deploy preview

https://deploy-preview-42236--material-ui.netlify.app/

Bundle size report

No bundle size changes (Toolpad)
No bundle size changes

Generated by 🚫 dangerJS against 7a5a5c7

Copy link
Member

@Janpot Janpot left a comment

Choose a reason for hiding this comment

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

Not sure about the file position. I hesitated to put this component inside a folder containing small UI components, but did not find a name for it

I'm thinking for the future we can collect them in a single module that contains building blocks for API pages, docs pages and landing pages. e.g. @mui/docs/pages, or @mui/docs/content,... In the meantime this is probably good enough. 👍

@alexfauquette alexfauquette merged commit 3c888ed into mui:next May 15, 2024
22 of 23 checks passed
@alexfauquette alexfauquette deleted the sync-section-titles branch May 15, 2024 08:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
scope: docs-infra Specific to the docs-infra product
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants