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] Fix 404 links #26963

Merged
merged 2 commits into from Jun 26, 2021
Merged

[docs] Fix 404 links #26963

merged 2 commits into from Jun 26, 2021

Conversation

oliviertassinari
Copy link
Member

We used to have none, It's getting us closer to the target.

Capture d’écran 2021-06-25 à 23 34 00

@oliviertassinari oliviertassinari added the docs Improvements or additions to the documentation label Jun 25, 2021
@mui-pr-bot
Copy link

mui-pr-bot commented Jun 25, 2021

No bundle size changes (experimental)

Generated by 🚫 dangerJS against 57de0ee

@@ -43,7 +43,7 @@ We avoid documenting native properties supported by the DOM like [`className`](/

### CSS Classes

All components accept a [`classes`](/customization/how-to-customize/#overriding-styles-with-classes) prop to customize the styles.
All components accept a `classes` prop to customize the styles.
Copy link
Member

Choose a reason for hiding this comment

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

I think we should rather evaluate what this section is for:

  1. do we still need it?
  2. where did the overriding-styles-with-classes section go
  3. why was the codeblock below removed

Again, please use accurate PR titles and description so that people can follow changes at a glance and not have to check each line changed to catch unrelated changes.

Copy link
Member Author

Choose a reason for hiding this comment

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

These changes were meant to be straightforward cleanup for outdated content.

  1. I don't know
  2. We have removed it, we don't want developers to use this approach.
  3. It's about makeStyles, which is legacy.

Copy link
Member Author

Choose a reason for hiding this comment

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

I have reverted, my intent was 404 fixex with no controversy on the changes. We can continue the discussion with @mnajdova

@oliviertassinari oliviertassinari merged commit 97bd233 into mui:next Jun 26, 2021
@oliviertassinari oliviertassinari deleted the docs-fix-404 branch June 26, 2021 10:50
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.

None yet

3 participants