Home button in navbar does not scroll to top on landing page #856

[divyanshmrawal]

Fixing - #856

Description

The issue where the Home button in the navbar was not functioning on the landing page has been fixed.

Previously, clicking the Home button did not scroll or redirect the user to the top of the page, resulting in broken navigation behavior. The problem was caused by missing or incorrect scroll/navigation handling for the Home button.

---

Fix Implemented

• Updated the Home button logic to correctly handle navigation.
• Ensured that clicking Home now scrolls the page back to the top of the landing page.
• Verified that the behavior works consistently after scrolling down the page.

---

Steps to Verify

1. Open the PictoPy landing page
2. Scroll down the page
3. Click on the Home button in the navbar

✅ The page now correctly scrolls to the top.

---

Result

• Navigation behavior is now consistent and intuitive.
• Improves overall user experience on the landing page.

---

Summary by CodeRabbit

• Style
  • Enhanced vertical spacing on key heading elements across landing pages for improved visual alignment and layout consistency.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

📝 Walkthrough

Walkthrough

Three landing page components receive minor CSS padding adjustments. Added bottom padding (pb-2 or pb-3) to heading elements across FAQ, Home1, and pictopy-landing pages, tightening vertical spacing without altering functionality or rendering behavior.

Changes

Cohort / File(s)	Summary
Heading spacing adjustments landing-page/src/Pages/FaqPage/FAQ.tsx, landing-page/src/Pages/Landing page/Home1.tsx, landing-page/src/Pages/pictopy-landing.tsx	Added bottom padding (pb-2 or pb-3) to heading elements to adjust vertical spacing; no changes to logic, state, or component behavior

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Suggested labels

UI, easy

Poem

🐰 Hop, hop, a spacing tweak so neat,
Padding added, spacing sweet,
Headings breathe with room below,
Minor tweaks that help things flow!

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	You can run @coderabbitai generate docstrings to improve docstring coverage.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The PR title accurately describes the main issue being fixed (Home button scroll-to-top functionality), which aligns with the actual changes described in the objectives, despite the changes involving only UI padding adjustments.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

📜 Recent review details

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 81286fa and 5d7da5c.

📒 Files selected for processing (3)

• landing-page/src/Pages/FaqPage/FAQ.tsx
• landing-page/src/Pages/Landing page/Home1.tsx
• landing-page/src/Pages/pictopy-landing.tsx

🔇 Additional comments (3)

landing-page/src/Pages/FaqPage/FAQ.tsx (1)

58-60: Minor styling adjustment looks fine, but this doesn't address the PR objective.

The pb-3 padding addition is a valid styling tweak. However, the PR claims to fix the Home button scroll-to-top behavior, yet this file only contains a CSS padding change with no navigation or scroll handling logic.

Please verify that the actual Home button scroll fix is included in this PR (likely in a navbar component not shown here).

landing-page/src/Pages/pictopy-landing.tsx (1)

59-61: Padding adjustment is acceptable.

The pb-3 class adds minor bottom spacing to the heading. This is a harmless UI tweak that aligns with the styling changes in the other files.

However, like the other files in this PR, this doesn't contain the Home button scroll-to-top fix described in the PR objectives.

landing-page/src/Pages/Landing page/Home1.tsx (1)

21-24: Styling change is fine.

The pb-2 padding addition provides consistent spacing below the heading, matching the styling pattern applied in the other files.

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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

[coderabbitai]

Actionable comments posted: 0

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)

landing-page/src/Pages/Landing page/Home1.tsx (1)

1-66: Scroll-to-top fix not implemented in provided files.

The PR description indicates a fix for Home button scroll-to-top behavior, but the reviewed file (Home1.tsx) contains only CSS padding adjustments. The landing-page Navbar component exists with a Home button that navigates to "/" without implementing scroll-to-top logic. Either the Navbar component needs to be updated with scroll-to-top functionality when clicking Home, or the PR description should be clarified.

📜 Review details

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 81286fa and 5d7da5c.

📒 Files selected for processing (3)

• landing-page/src/Pages/FaqPage/FAQ.tsx
• landing-page/src/Pages/Landing page/Home1.tsx
• landing-page/src/Pages/pictopy-landing.tsx

🔇 Additional comments (3)

landing-page/src/Pages/FaqPage/FAQ.tsx (1)

58-60: Minor styling adjustment looks fine, but this doesn't address the PR objective.

The pb-3 padding addition is a valid styling tweak. However, the PR claims to fix the Home button scroll-to-top behavior, yet this file only contains a CSS padding change with no navigation or scroll handling logic.

Please verify that the actual Home button scroll fix is included in this PR (likely in a navbar component not shown here).

landing-page/src/Pages/pictopy-landing.tsx (1)

59-61: Padding adjustment is acceptable.

The pb-3 class adds minor bottom spacing to the heading. This is a harmless UI tweak that aligns with the styling changes in the other files.

However, like the other files in this PR, this doesn't contain the Home button scroll-to-top fix described in the PR objectives.

landing-page/src/Pages/Landing page/Home1.tsx (1)

21-24: Styling change is fine.

The pb-2 padding addition provides consistent spacing below the heading, matching the styling pattern applied in the other files.