Conversation
There was a problem hiding this comment.
Pull request overview
Updates the hackers hub hero experience by introducing a new “hacking” hero layout with new illustration assets and a redesigned countdown, while also cleaning up related navigation code.
Changes:
- Added a new
HeroHackingcomponent (with new hero SVG assets) and wired it into the hackers hub page. - Introduced a new countdown component for the hacking hero (and a new TimeTracker component file).
- Refactored
Navbarquery param handling and removed commented-out/unused code.
Reviewed changes
Copilot reviewed 5 out of 11 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| public/hackers/hero/StarRight.svg | Adds new hero decorative star asset. |
| public/hackers/hero/StarLeft.svg | Adds new hero decorative star asset. |
| public/hackers/hero/PeepingFrog.svg | Adds new hero illustration asset. |
| public/hackers/hero/PeepingDuck.svg | Adds new hero illustration asset. |
| public/hackers/hero/PeepingCow.svg | Adds new hero illustration asset. |
| public/hackers/hero/PeepingBunny.svg | Adds new hero illustration asset. |
| app/(pages)/_components/Navbar/Navbar.tsx | Updates navbar query param handling and simplifies code. |
| app/(pages)/(hackers)/_components/HomeHacking/_components/TimeTracker.tsx | Adds a time-tracking/countdown helper component. |
| app/(pages)/(hackers)/_components/HomeHacking/_components/Countdown.tsx | Adds the hacking hero countdown UI and logic. |
| app/(pages)/(hackers)/_components/HomeHacking/HeroHacking.tsx | Implements the new hacking hero layout and countdown integration. |
| app/(pages)/(hackers)/(hub)/page.tsx | Swaps the hub hero from HeroMVP to HeroHacking. |
| app/(pages)/(hackers)/(hub)/layout.tsx | Wraps the protected layout contents in a div (structural change only). |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 4 out of 10 changed files in this pull request and generated 7 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
michelleyeoh
left a comment
michelleyeoh
left a comment
There was a problem hiding this comment.
Nice work!
Thanks for fixing up navbar + making hero easy to later import gifs
Try to monitor vs code terminal next time, the useEffect (now removed) was spamming POST calls
Fixed up some legacy code - made rollouts easier to read from page.tsx
Removed weird shadow and border & renamed hero file
No description provided.