-
Notifications
You must be signed in to change notification settings - Fork 595
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
#ISSUE-460 used flexbox instead of hard coded values and made search bar sticky #444
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
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 job, @parth305! 🎉 Thank you for opening a pull request. Your contribution is valuable and we appreciate your efforts to improve our project.
Soon the maintainers/owner will review it and provide you with feedback/suggestions.
Hey @parth305 I appreciate your contribution but first you have to open an issue than start working on it. I like the changes but scroll to top has stopped working, when user click next than scroll to top should take place, look into it and update the PR and also open an issue for it. |
Hello @shyamtawli i have created issue and fixed the changes you have asked. please review my PR and merge it with master. |
Migrated to TailwindCSS PR #852 |
Description
used flex box instead f hard coded values of margin and widths
Changes Proposed
i have changed css to use flex box to style components. search bar and cards where floating over sidebar so i have tried to fix that with flexbox, and i have made search bar sticky.
Checklist