Skip to content

feat(apply): improve hero section typography and spacing #584

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.

Sign up for GitHub

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

Closed
LogicVortex123 wants to merge 1 commit into from
Closed

feat(apply): improve hero section typography and spacing #584

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
13 changes: 11 additions & 2 deletions src/pages/apply.jsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -68,8 +68,17 @@ export default function About() {
<Container.Outer className="mt-20 mb-28 ">
<div className='overflow-x-hidden'>
<div className="mt-5 mx-4 md:mx-16">
<h5 className='font-bold text-2xl text-zinc-600 dark:text-zinc-400 text-lg font-mono leading-7'> Learn how to apply for an <span className="text-[#3C982C] dark:text-yellow-400">opportunity</span> to work on open-source projects and gain<span className="text-[#3C982C] dark:text-yellow-400"> real-world experience</span> through Google Summer of Code.</h5>
<p className='text-zinc-600 mt-4 dark:text-zinc-400 text-lg font-mono leading-7'>
<h1 className="font-bold text-4xl sm:text-5xl md:text-6xl
text-yellow-400 mb-12 leading-tight">
Start Your Open-Source Journey
</h1>
<h2 className="mt-4 font-mono font-medium
text-lg sm:text-xl md:text-2xl
text-zinc-600 dark:text-zinc-400
leading-relaxed">
Learn how to apply for an <span className="text-yellow-400 font-bold">opportunity</span> to work on open-source projects and gain <span className="text-yellow-400 font-bold">real-world experience</span> through Google Summer of Code.
</h2>
<p className='mt-6 text-zinc-600 mt-4 dark:text-zinc-400 text-lg font-mono leading-7'>
Copy link
Copy Markdown

@coderabbitai coderabbitai Bot Feb 17, 2026

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟡 Minor

Remove duplicate margin-top class.

The className contains both mt-6 and mt-4. In Tailwind CSS, when duplicate utility classes are present, only the last one takes effect. This creates confusion about the intended spacing.

🧹 Proposed fix to remove the duplicate 
-            <p className='mt-6 text-zinc-600 mt-4  dark:text-zinc-400 text-lg font-mono leading-7'>
+            <p className='mt-6 text-zinc-600 dark:text-zinc-400 text-lg font-mono leading-7'>

Note: I kept mt-6 as it provides more spacing from the h2 above. If mt-4 was intentional, remove mt-6 instead.

📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
<p className='mt-6 text-zinc-600 mt-4 dark:text-zinc-400 text-lg font-mono leading-7'>
<p className='mt-6 text-zinc-600 dark:text-zinc-400 text-lg font-mono leading-7'>
🤖 Prompt for AI Agents 
Verify each finding against the current code and only fix it if needed.

In `@src/pages/apply.jsx` at line 81, Duplicate margin-top Tailwind utilities
exist on the paragraph element with className 'mt-6 text-zinc-600 mt-4 
dark:text-zinc-400 text-lg font-mono leading-7'; remove the redundant class
(delete either mt-4 or mt-6) so only one mt-* remains — keep mt-6 as suggested —
by editing the JSX <p> element's className to eliminate the duplicate.

Our application timeline is your step-by-step guide to becoming a part of our open-source community and contributing to projects that are shaping the future of technology. From submitting your proposal to final evaluations, we&apos;ll walk you through the process every step of the way. Don&apos;t miss this opportunity to be a part of something great and apply now!
</p>
</div>
Expand Down