Skip to content

Conversation

oliviertassinari
Copy link
Member

This is following #2382.
We now have to convert every doc component. That's pretty straightforward.
This has several advantages:

  • Property description are no longer duplicated. That reduce the cost of maintaining them and the risk of inconsistency.
  • Property description are right into the source code. The improve the explicitness.
  • Examples code is no longer duplicated, with the same advantages as with the property description
  • Examples are explicit and almost completly decoupled from the doc source.
  • Those new examples are going to be great component to write unit test for
  • The writing of the doc is simplified by using the markdown abstraction over the jsx.
  • The code highlighting is far better.

@shaurya947
Copy link
Contributor

Looks good to me. So happy with where the docs are going ^_^

src/avatar.jsx Outdated
Copy link
Collaborator

Choose a reason for hiding this comment

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

  • The icon or the letter's color.

@alitaheri
Copy link
Collaborator

@oliviertassinari If you be so kind, let's address #2310 while we are migrating the docs. Add a className to Avatar while you're at it 😁. Like I did with #2415

src/avatar.jsx Outdated
Copy link
Collaborator

Choose a reason for hiding this comment

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

I don't mind this, but I think it helps readability to be more explicit. If you agree please add className to props destructuring and to the div's props.

@alitaheri
Copy link
Collaborator

after that do a rebase and merge. 👍 👍 👍

oliviertassinari added a commit that referenced this pull request Dec 8, 2015
[Doc] Convert Avatar to use the new standard
@oliviertassinari oliviertassinari merged commit a5c89aa into mui:master Dec 8, 2015
@oliviertassinari oliviertassinari deleted the doc-codgen-avatar branch December 8, 2015 08:57
@alitaheri
Copy link
Collaborator

🎉 👍

@zannager zannager added docs Improvements or additions to the documentation. scope: avatar Changes related to the avatar. labels Mar 14, 2023
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. scope: avatar Changes related to the avatar.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants