-
-
Notifications
You must be signed in to change notification settings - Fork 590
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
added footer #160
added footer #160
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.
Also, can we make background blur
instead of using this solid
color?
maybe what we can do is instead of keeping it fixed, I add the footer static inside the right container and make cards container have a fixed height just above to the footer |
Okay |
fixed overflow, made cards container overflow-scroll instead of whole container scrolling
closes #161 as well |
here's how it looks on mine screen-capture.17.webm |
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 Aadarsh. But having 2 small issues:
- It's showing 2 scrollbars
- Ig it should not be fixed this way, what i meant is when user reaches the end of the page, then we should display this footer. And for initial state (when no category is selected) it should be at the bottom of the page just like it's now.
And thanks for this 😊 |
I actually fixed the bigger scrollbar, that's exactly what is being addressed in this issue #161, I'll look on it again must be because of smaller screen size ig |
@Aadarsh805 I guess not for desktop but we can fix it t top for other devices(phone, tabs). Cz' we have sidebar in desktop so there's no need for it necessarily |
Updated screen-capture.19.webm |
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.
Thank you so much Aadarsh, appreciate it. Now it looks perfect.
Fixes Issue
Closes #117
Screenshots
Note to reviewers
Suggest changes if any