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

added footer #160

Merged
merged 6 commits into from
Feb 11, 2023
Merged

added footer #160

merged 6 commits into from
Feb 11, 2023

Conversation

Aadarsh805
Copy link
Contributor

Fixes Issue

Closes #117

Screenshots

Screenshot from 2023-02-09 10-26-33

Note to reviewers

Suggest changes if any

@vercel
Copy link

vercel bot commented Feb 9, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated
linkshub ✅ Ready (Inspect) Visit Preview 💬 Add your feedback Feb 11, 2023 at 9:23AM (UTC)

Copy link
Owner

@rupali-codes rupali-codes left a 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?

@Aadarsh805
Copy link
Contributor Author

Also, can we make background blur instead of using this solid color?

something like this?
108 (3)

@rupali-codes
Copy link
Owner

Hmm.

Also, can we make background blur instead of using this solid color?

something like this? 108 (3)

Looks good tho, but wud hv been better if cud just make it within right container. If not, just commit the changes

@Aadarsh805
Copy link
Contributor Author

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
something like that
let me try

@rupali-codes
Copy link
Owner

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 something like that let me try

Okay

fixed overflow,
made cards container overflow-scroll instead of whole container scrolling
@Aadarsh805
Copy link
Contributor Author

closes #161 as well

@Aadarsh805
Copy link
Contributor Author

here's how it looks on mine

screen-capture.17.webm

Copy link
Owner

@rupali-codes rupali-codes left a 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:

  1. It's showing 2 scrollbars

image

  1. 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.

@rupali-codes
Copy link
Owner

closes #161 as well

And thanks for this 😊

@Aadarsh805
Copy link
Contributor Author

Great work Aadarsh. But having 2 small issues:

  1. It's showing 2 scrollbars

image

  1. 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.

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
and I'll try to implement it the other way as you suggested
One question, do we want the top bar fixed? i suggest it should be

@rupali-codes
Copy link
Owner

@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

@Aadarsh805
Copy link
Contributor Author

Updated

screen-capture.19.webm

Copy link
Owner

@rupali-codes rupali-codes left a 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.

@rupali-codes rupali-codes merged commit 509e112 into rupali-codes:main Feb 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feature] add footer at bottom of the site
2 participants