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] Add guides to overriding component structure in Base UI and Joy UI docs #34990

Merged
merged 31 commits into from
Apr 12, 2023

Conversation

samuelsycamore
Copy link
Member

@samuelsycamore samuelsycamore commented Nov 2, 2022

Part of #34968

This PR adds a doc on "Overriding component structure" to the How To Guide sections of the MUI Base and Joy UI docs.

As I was writing the main text, I realized that it only really makes sense if you have an accurate mental model of the components and understand the terminology we use, so I tried to introduce these concepts as succinctly as possible in the intro.

The content of the two docs is essentially the same. I added a link to the Joy UI page to explain that Joy will automatically correct semantically inappropriate HTML.

We could also add this guide to the Material UI docs, but these props are not consistently available to components in that library (yet), so I wasn't sure how to approach this. Maybe it's best to wait to introduce these props in the Material UI docs.

Ideally these docs will describe everything that a user needs to know on this topic. Once they're published, then we can decide whether or not to move/remove these details from the Component pages in Base and Joy where they're present.

@samuelsycamore samuelsycamore added docs Improvements or additions to the documentation package: base-ui Specific to @mui/base package: joy-ui Specific to @mui/joy labels Nov 2, 2022
@mui-bot
Copy link

mui-bot commented Nov 2, 2022

@siriwatknp
Copy link
Member

siriwatknp commented Nov 4, 2022

@hbjORbj is working on the components => slots for Joy UI, so I think you can move forward with the slots in this PR.

@samuelsycamore samuelsycamore marked this pull request as ready for review January 26, 2023 22:43
@github-actions github-actions bot removed the PR: out-of-date The pull request has merge conflicts and can't be merged label Apr 5, 2023
@samuelsycamore samuelsycamore requested review from siriwatknp and michaldudak and removed request for mnajdova and michaldudak April 5, 2023 17:10
@samuelsycamore samuelsycamore changed the title [docs] Add guides to overriding component structure in MUI Base and Joy UI docs [docs] Add guides to overriding component structure in Base UI and Joy UI docs Apr 5, 2023
@samuelsycamore
Copy link
Member Author

I went ahead and changed "MUI Base" to "Base UI" in this doc.

@github-actions github-actions bot added the PR: out-of-date The pull request has merge conflicts and can't be merged label Apr 5, 2023
@github-actions github-actions bot removed the PR: out-of-date The pull request has merge conflicts and can't be merged label Apr 5, 2023
@github-actions github-actions bot added the PR: out-of-date The pull request has merge conflicts and can't be merged label Apr 6, 2023
@github-actions github-actions bot removed the PR: out-of-date The pull request has merge conflicts and can't be merged label Apr 6, 2023
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.

👍 Looks great

…-component-structure.md

Co-authored-by: Siriwat K <siriwatkunaporn@gmail.com>
Signed-off-by: Sam Sycamore <71297412+samuelsycamore@users.noreply.github.com>
return (
<Button
component="a"
href="https://mui.com/about-us/"
Copy link
Member

Choose a reason for hiding this comment

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

This is a dead link. Fixed in #36969.

@samuelsycamore samuelsycamore deleted the overriding-docs branch April 3, 2024 17:24
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 package: base-ui Specific to @mui/base package: joy-ui Specific to @mui/joy
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants