Skip to content

feat: improve subscription section visibility and layout#635

Open
Kashish-1105 wants to merge 1 commit into
GitMetricsLab:mainfrom
Kashish-1105:improve-subscription-section
Open

feat: improve subscription section visibility and layout#635
Kashish-1105 wants to merge 1 commit into
GitMetricsLab:mainfrom
Kashish-1105:improve-subscription-section

Conversation

@Kashish-1105
Copy link
Copy Markdown

@Kashish-1105 Kashish-1105 commented May 31, 2026

Summary

This PR enhances the footer subscription section to improve visibility, engagement, and overall visual balance.

Changes Made

  • Increased the prominence of the subscription section
  • Added a stronger heading and supporting text
  • Improved spacing and layout structure
  • Enhanced CTA button visibility and hierarchy
  • Added better visual separation while maintaining the existing design language
  • Improved responsiveness across mobile, tablet, and desktop devices
  • Preserved existing functionality and theme compatibility

Testing

  • Verified UI locally
  • Tested responsiveness across different viewport sizes
  • Confirmed dark/light mode compatibility
  • Successfully completed production build using npm run build

Files Modified

  • src/components/Footer.tsx

Screenshots

Before

before

After

after

Summary by CodeRabbit

  • Style
    • Redesigned newsletter subscription section with an enhanced card-based layout, gradient background, and improved visual hierarchy
    • Adjusted footer layout for better column alignment
    • Updated subscription button label to "Subscribe Now"

@netlify
Copy link
Copy Markdown

netlify Bot commented May 31, 2026

Deploy Preview for github-spy ready!

Name Link
🔨 Latest commit c451fb2
🔍 Latest deploy log https://app.netlify.com/projects/github-spy/deploys/6a1c44ad1a729e0009be500c
😎 Deploy Preview https://deploy-preview-635--github-spy.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

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

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 31, 2026

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 6e3ef1b6-5d07-4631-817d-4de17959b998

📥 Commits

Reviewing files that changed from the base of the PR and between 53f820b and c451fb2.

📒 Files selected for processing (1)
  • src/components/Footer.tsx

📝 Walkthrough

Walkthrough

Footer component redesigned with brand column reduced to accommodate wider newsletter section. Newsletter area transformed from a compact inline form to a prominent card layout with gradient background, enhanced typography, updated input/button styling, and button label changed to "Subscribe Now".

Changes

Footer Layout and Newsletter Redesign

Layer / File(s) Summary
Footer layout and newsletter section redesign
src/components/Footer.tsx
Brand column responsive span reduced from lg:col-span-5 to lg:col-span-4. Newsletter section completely redesigned with card-style layout, new gradient background layer, updated heading and description text, revised Tailwind-based input and button styling, and submit button label changed from "Subscribe" to "Subscribe Now" while maintaining handleSubscribe form submission behavior.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly related issues

Possibly related PRs

Suggested labels

level:intermediate, quality:clean

Poem

🐰 A footer redesigned with care,
Grid columns shuffled with flair,
Newsletter blooms in a card so bright,
Gradient glow and "Subscribe Now"—pure delight!

🚥 Pre-merge checks | ✅ 3 | ❌ 2

❌ Failed checks (1 warning, 1 inconclusive)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
Description check ❓ Inconclusive The description includes a summary, changes made, testing performed, and screenshots, but does not follow the required template structure with sections like 'Related Issue' or 'Type of Change'. Consider restructuring the description to match the repository template with proper sections including 'Related Issue', 'Type of Change' checkboxes, and standard formatting for consistency.
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately summarizes the main change—improving subscription section visibility and layout—which aligns with the actual modifications made to the Footer component.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

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

🎉 Thank you @Kashish-1105 for your contribution. Please make sure your PR follows https://github.com/GitMetricsLab/github_tracker/blob/main/CONTRIBUTING.md#-pull-request-guidelines

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant