-
Notifications
You must be signed in to change notification settings - Fork 2
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
Sponsors #12
Sponsors #12
Conversation
Your Render PR Server URL is https://posua-2024-pr-12.onrender.com. Follow its progress at https://dashboard.render.com/static/srv-cnum9pfjbltc73cdl3a0. |
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 mapping for similar div items instead of making separate div containers for each. This reduces redundancy and improves code readability.
src/Pages/Sponsors/sponsors.scss
Outdated
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.
Focus a lot on the mobile view responsiveness part. Lots of overflow issues are there.
</div> | ||
<div> | ||
<div className="heading"> | ||
<img |
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.
Do not make the heading a complete image itself. At least the writing part, and also the Japis could be made rotatable
added sponsor page, yet to be made responsive