visgl / deck.gl / 30503749877

84%

push

github

feat(widgets): custom tooltip system for widget buttons (#10435)

* feat(widgets): add custom tooltip system for widget buttons

Replace native HTML title tooltips with themed, positioned tooltips using
a Preact Tooltip component and @floating-ui/dom. Zero changes to core.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* fix(widgets): use firstElementChild for tooltip positioning

display: contents elements have no bounding box, so computePosition
must reference the actual child button element. Also switch to
pointerover/pointerout which bubble through pointer-events: none parents.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* fix(widgets): fix tooltip positioning, button group styling, and instant show

- Fix vertical centering by using inline critical styles (whiteSpace/fontSize/padding)
  so computePosition gets correct dimensions regardless of stylesheet load timing
- Fix button group border-radius by using descendant selectors for button
- Show tooltips instantly with opacity transition to prevent flash
- Use ref callback instead of useEffect for immediate positioning

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* feat(widgets): support `false` to explicitly disable tooltips

Tooltip props now accept `string | HTMLElement | false`:
- `false` explicitly disables the tooltip (no fallback to label)
- `undefined` (default) falls back to the widget's label
- Update docs to document the `false` option

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* fix(widgets): use pointerEnter/Leave and correct accessibility docs

Switch from pointerOver/Out to pointerEnter/Leave to prevent spurious
tooltip flicker when moving between child elements. Remove unimplemented
aria-describedby and :focus-visible claims from docs.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* fix(widgets): update tests for tooltip system (title → aria-label)

Tests were checking button.title which no longer exists after tooltip... (continued)

8260 of 10441 branches covered (79.11%)

Branch coverage included in aggregate %.

44 of 46 new or added lines in 7 files covered. (95.65%)

14618 of 17084 relevant lines covered (85.57%)

18767.31 hits per line