fix: tweak above the fold wording

[jobenjada]

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

2 Ignored Deployments

Name	Status	Preview	Comments	Updated (UTC)
formbricks-cloud	⬜️ Ignored (Inspect)	Visit Preview		Dec 29, 2023 0:47am
formbricks-com	⬜️ Ignored (Inspect)	Visit Preview		Dec 29, 2023 0:47am

[github-actions]

Thank you for following the naming conventions for pull request titles! 🙏

[review-agent-prime]

apps/formbricks-com/components/shared/Header.tsx

Listening to the scroll event can be expensive in terms of performance, especially if the event handler performs complex operations. In this case, the event handler is simple, but it's still a good practice to throttle or debounce the event to reduce the number of times the handler is called.
Create Issue
See the diff
Checkout the fix

    useEffect(() => {
      const handleScroll = throttle(() => {
        if (window.scrollY > 250) {
          setStickyNav(true);
        } else {
          setStickyNav(false);
        }
      }, 200);
      window.addEventListener("scroll", handleScroll);
      return () => {
        window.removeEventListener("scroll", handleScroll);
      };
    }, []);

git fetch origin && git checkout -b ReviewBot/Impro-yuh6ni4 origin/ReviewBot/Impro-yuh6ni4

It's a good practice to provide a skip link at the beginning of the page. This allows keyboard users to skip directly to the main content, bypassing the header navigation.
Create Issue
See the diff
Checkout the fix

    return (
      <>
        <a href="#maincontent" className="sr-only focus:not-sr-only">
          Skip to main content
        </a>
        <Popover className={`${stickyNavClass}`} as="header">
          ...
        </Popover>
      </>
    );

git fetch origin && git checkout -b ReviewBot/Impro-et7bta9 origin/ReviewBot/Impro-et7bta9