-
Notifications
You must be signed in to change notification settings - Fork 0
componentRoute #1
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
base: main
Are you sure you want to change the base?
Conversation
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.
Great work.
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.
Amazing work 🤯,I like your approach of putting components in array and loop over it when clicked in the view,I didn't find any such major changes in code,just added some small suggestions
border: "1px solid rgba(31, 41, 55)", | ||
navHover: "#131314", | ||
highlightBcg: "#26272b", | ||
codeBcg: "#222328", |
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.
use variables instead hard coding,here repetitions of numerical value is seen
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.
Its not exactly hard coded. Its the theme that I am providing to the context and they are reused everywhere like backgroundColor: theme.highlightBcg
PARS.css Component Library
Dev Preview: https://pars.netlify.app/
Open for PR review @Team-Asynchronous @vaishnavme @amansethi00
Things to do: