Skip to content

Conversation

@maxn990
Copy link
Member

@maxn990 maxn990 commented Sep 8, 2025

ℹ️ Issue

Closes no ticket

📝 Description

  1. Removed extra space from jumpstart page
  2. Added additional space to mobile pages where the nav bar overlapped with the page content

✔️ Verification

Visually checked every page on mobile and desktop

🏕️ (Optional) Future Work / Notes

Did you notice anything ugly during the course of this ticket? Any bugs, design challenges, or unexpected behavior? Write it down so we can clean it up in a future ticket!

@maxn990 maxn990 requested a review from amywng September 8, 2025 17:50
Copy link
Member

@amywng amywng left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you also add more space between the intro and the cards, and have less space between the cards and timeline

<div className="flex flex-row items-center overflow-x-hidden">
<div className="flex flex-col h-[100vh] md:h-[90vh] justify-center md:mr-10 md:w-2/3 relative">
<div className="flex flex-col justify-center md:mr-10 md:w-2/3 relative py-10 md:py-20">
<p className="text-2xl md:text-4xl text-[#333333] md:mb-3 font-medium">
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can we have the title match the other pages so text-2xl md:text-5xl

<div className="absolute top-[1%] md:top-0 left-0 w-[100%] md:w-[60%] aspect-square bg-purple-radial-gradient-left z-[-1] "></div>
<div className="absolute top-1/3 right-0 w-[100%] md:w-[60%] aspect-square bg-pink-radial-gradient-right z-[-1] "></div>
<div className="w-[90%] md:max-w-[1100px]">
<div className="mt-24 md:mt-0 w-[90%] md:max-w-[1100px]">
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The top padding still doesn't match the other pages for me?

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.

3 participants