-
-
Notifications
You must be signed in to change notification settings - Fork 103
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: rework about and home page to mention this is the community fork #217
Conversation
…sion, and minor changes - Added the community as part of the team, by explaining why we forked the repo and by adding a dynamic image that show the top 24 contributors. Prob is that it don't sort by team members but rather by commit numbers - Added a message in the index page that says this version is a fork - Changeed the catchphrase in the homepage - Rephrased a sentence inside the introduction grid shown in the homepage and grid section fix #194
I took more than 1h30 to write that. It's very difficult to phrase things well in order to stay the less subjective possible, so I hope everything is fine. Anyway, feel free to leave reviews. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, I am Not listed in the members :(
@wuda-io the URL lists / shows only the top 24 contributors (commits) who contributed commits to the main branch. https://contrib.rocks/image?repo=materializecss/materialize&max=50 shows the top 50 contributors. See also https://github.com/materializecss/materialize/graphs/contributors |
Yes I have to make more commits 😜 |
Thanks for the reviews I will correct this tonight |
ready to be merged @materializecss/members-write-access |
Proposed changes
dynamic image that show the top 24 contributors. Prob is that it don't sort by team members but
rather by commit numbers
fix #194
Screenshots (if appropriate) or codepen:
Click here to see screenshots & changes:
Types of changes
Checklist: