Hp Deep: EventList Page and Overall Theming

[HarshHp13]

No description provided.

[HarshHp13]

EventList Page and overall theming

[shubhanshu02]

Great work @HarshHp13 and @ideepverma 🙌 🔥! The website looks great on the dark theme.

Just a few suggestions on my side regarding the Navbar and theming.

• Consider the case where the user scrolls to some part of the page where the text on the page overlaps with text on the navbar. Here, both are not readable and seem a bit odd. To give an example, see below.
  

• For the light theme, the colors are kind of too bright. I guess we need to discuss with the design team to work out on colors.
  

• Homepage: The "Sponsor Broucher" and "Sponsor Us" buttons overlap on smaller screen widths like iPhone 12 Pro.
  

[shubhanshu02]

Regarding the event details page, I'll review in some time

[shubhanshu02]

For the page /events

• The links for individual events are not added to the event, even though the entire div is acting as a button. (In older versions of the website, this will point to the individual event page. E.g. /events/ctf). This is working fine for the cards on mobile devices.
• Width of the following card is fixed to 796px, instead use a relative size.
  
• The background color of image in the card for individual events needs to be changed to a better background.
  

[netlify]

❌ Deploy Preview for codefest-frontend failed.

Name	Link
🔨 Latest commit	3a730a4
🔍 Latest deploy log	https://app.netlify.com/sites/codefest-frontend/deploys/632eded5cf5a260009a53ac0

[netlify]

✅ Deploy Preview for codefest-frontend ready!

Name	Link
🔨 Latest commit	4087008
🔍 Latest deploy log	https://app.netlify.com/sites/codefest-frontend/deploys/64e11ed8298c5d00083a403e
😎 Deploy Preview	https://deploy-preview-2--codefest-frontend.netlify.app/
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site configuration.