Skip to content

Conversation

maoryadin
Copy link
Owner

@maoryadin maoryadin commented Mar 25, 2025

Refactor componentDidUpdate method in src/components/Overlay.tsx to handle spotlightClicks, disableOverlay, and lifecycle changes.

  • Add logic to remove mousemove event listener and reset mouseOverSpotlight state when spotlightClicks or lifecycle changes.
  • Add logic to add mousemove event listener when spotlightClicks is true and lifecycle is LIFECYCLE.TOOLTIP.
  • Remove redundant mousemove event listener removal logic.

For more details, open the Copilot Workspace session.

…lifecycle changes in componentDidUpdate

Refactor `componentDidUpdate` method in `src/components/Overlay.tsx` to handle `spotlightClicks`, `disableOverlay`, and `lifecycle` changes.

* Add logic to remove `mousemove` event listener and reset `mouseOverSpotlight` state when `spotlightClicks` or `lifecycle` changes.
* Add logic to add `mousemove` event listener when `spotlightClicks` is true and `lifecycle` is `LIFECYCLE.TOOLTIP`.
* Remove redundant `mousemove` event listener removal logic.

---

For more details, open the [Copilot Workspace session](https://copilot-workspace.githubnext.com/maoryadin/react-joyride?shareId=XXXX-XXXX-XXXX-XXXX).
@maoryadin maoryadin merged commit b6fca1c into fix-overlay-spotlight Mar 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant