Initialize BackToTop visibility on mount

[Copilot]

BackToTop never called handleScroll() on mount, so the button stayed hidden if the page loaded already scrolled past 400px — requiring a manual scroll event to trigger visibility.

Change

• Call handleScroll() once immediately after registering the scroll listener to seed the initial visible state:

useEffect(() => {
  const handleScroll = () => setVisible(window.scrollY > 400);
  window.addEventListener("scroll", handleScroll, { passive: true });
  handleScroll(); // initialize on mount
  return () => window.removeEventListener("scroll", handleScroll);
}, []);

Original prompt

---

This section details on the original issue you should resolve

<issue_title>Initialize BackToTop visibility on mount by calling handleScroll() once</issue_title>
<issue_description>Source: Copilot review comment on PR #8 — src/Components/BackToTop.jsx line 42

Problem

BackToTop never calls handleScroll() on mount, so if the page loads already scrolled (>400px) the button will remain hidden until the user scrolls again.

Suggested Fix

Call handleScroll() once after registering the listener to initialize visible correctly:

useEffect(() => {
  window.addEventListener('scroll', handleScroll);
  handleScroll(); // initialize on mount
  return () => window.removeEventListener('scroll', handleScroll);
}, []);

Review thread: #8 (comment)</issue_description>

Comments on the Issue (you are @copilot in this section)

• Fixes Initialize BackToTop visibility on mount by calling handleScroll() once #17

---

💬 Send tasks to Copilot coding agent from Slack and Teams to turn conversations into code. Copilot posts an update in your thread when it's finished.

[Yasar2019]

Good

[github-actions]

Azure Static Web Apps: Your stage site is ready! Visit it here: https://witty-mud-031b23a0f-20.eastus2.2.azurestaticapps.net

[Yasar2019]

Good