Skip to content

Fix tooltips blocking clicks after hide#21

Merged
ChrisHuynh333 merged 2 commits into
mainfrom
fix/pathogen-tooltips-overlap
Apr 28, 2026
Merged

Fix tooltips blocking clicks after hide#21
ChrisHuynh333 merged 2 commits into
mainfrom
fix/pathogen-tooltips-overlap

Conversation

@joshsadam
Copy link
Copy Markdown
Contributor

@joshsadam joshsadam commented Apr 28, 2026

What does this PR do and why?

Tooltips were fading out visually but could still intercept pointer events, which meant “closed” tooltips could block clicks/hover on underlying UI. Added a lookbook preview at the bottom of the Tooltips that allows you to test to make sure this is working.

This PR:

  • Adds a timeout-based hide step after the fade-out transition so the tooltip is fully hidden after closing.
  • Updates tooltip CSS so pointer events are disabled when hidden and enabled when visible.
  • Adds Vitest coverage to ensure tooltips show/hide correctly and don’t block interactions once closed.

Screenshots or screen recordings

N/A

How to set up and validate locally

  1. Check out fix/pathogen-tooltips-overlap.
  2. Install JS deps: pnpm install.
  3. Run unit tests: pnpm test:unit (or pnpm test).

PR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

- Introduce a timeout mechanism to hide tooltips after a fade-out transition, ensuring closed tooltips do not block interactions.
- Update CSS to manage pointer events, allowing tooltips to be non-interactive when hidden and interactive when visible.
- Add tests to verify the new hiding behavior and ensure tooltips are correctly shown and hidden based on user interactions.
@joshsadam joshsadam changed the title Enhance tooltip behavior and styling for improved interaction Fix tooltips blocking clicks after hide Apr 28, 2026
@joshsadam joshsadam self-assigned this Apr 28, 2026
- Introduce a new visual test to verify that tooltips do not block clicks on underlying elements after they are hidden.
- Update tooltip preview files to include a new method for testing this behavior.
- Refactor existing HTML templates to improve consistency and styling across tooltip previews.
@joshsadam joshsadam requested a review from ChrisHuynh333 April 28, 2026 20:50
@ChrisHuynh333 ChrisHuynh333 merged commit c5ba9b7 into main Apr 28, 2026
3 checks passed
@ChrisHuynh333 ChrisHuynh333 deleted the fix/pathogen-tooltips-overlap branch April 28, 2026 21:27
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.

2 participants