Duplicated scroll logic and inconsistent scroll behavior

[Harsh-goswami-103]

Problem

scrollToSection() is copy-pasted into Navbar.tsx, Hero.tsx, and Pricing.tsx. Separately, FAQ/FinalCTA/Footer navigate via raw <a href="#contact"> anchors. So some CTAs do JS smooth scrolling while others do an instant native jump to the same targets.

Impact

• Inconsistent UX (smooth vs. jump) for buttons that look identical.
• DRY/maintainability problem — behavior changes must be made in multiple places.

Acceptance Criteria

• Extract a single scrollToSection helper (or a <SmoothLink> component).
• Use one consistent navigation mechanism across all CTAs.

Priority

Low