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

Updated site page and added few additional css class #20

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

rinturaj
Copy link

Update:
SIte: Home page redesigned
SCSS: Added new style for card and fonts

@rinturaj
Copy link
Author

i would like to contribute this light weight material 3 repo.

@robinlahtinen robinlahtinen added documentation Improvements or additions to documentation enhancement New feature or request labels Jun 29, 2022
@robinlahtinen
Copy link
Member

Awesome to see a contribution on a extremely work-in-progress project! I were not expecting any outside contributions at this stage. I'm planning to tidy up the codebase and develop more features and better documentation before actively sharing this project with the open source community.

I'll start reviewing the pull request when I have time. From a quick glance and testing out the updated site it looks good, but code-wise there are some linting errors and some added CSS classes could be moved to their own file for the documentation site. Reminder to me and the project to make better contributor documentation and commenting the code better.

I'm not planning to add more contributors to the @mdlightdev organization until the projects has seen some growth and activity warranting for more lead team members. Contributing through pull requests is encouraged and I'll start pushing commits as pull requests also when I'm done with what I see as a initial development phase.

@robinlahtinen robinlahtinen self-requested a review June 29, 2022 20:35
@rinturaj
Copy link
Author

Thanks for the update @robinlahtinen .

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants