Files
gui/apps/kitchen-sink/tests/PopoverHoverable.test.tsx
zeekayandHanzo Dev 69c6e860d7 transplant onto tamagui v2.6.0, brand as a re-runnable transform
This repo could never sync. There was NO merge-base with tamagui — 372 commits
here against 15,791 there, unrelated histories — because it began as a
copy-paste rebrand with a flattened layout. Every "sync" since was a hand
re-application that drifted, which is why the tree sat on 7.3.1 while upstream
shipped fixes we then rediscovered by hand.

This commit's PARENT IS TAMAGUI. A future sync is `git merge tama/main` then
`node scripts/rebrand.mjs` — a real merge, because the history is now shared.

174 packages: 166 upstream, our 8 restored (chrome, shell, telemetry, recipes,
get, font-geist-{sans,mono}, @hanzo/booker), bento-get gained. 170/170 build.

Our patches to files UPSTREAM ALSO OWNS are carried, which the package-level
delta missed and which was where the value actually was:
  - useMedia's server snapshot (`initState ?? getMedia()`). Upstream still
    returns undefined there, so every Tamagui Dialog kills a prerender — the
    exact crash hanzo.app spent a day on.
  - data-slot on web host nodes, absent upstream entirely.
Both applied ONTO upstream's newer files (it rewrote useMedia +150/-63 and
createComponent +160/-55), marked `PATCH (ours, carried across the transplant)`.

Seven traps are encoded in scripts/rebrand.mjs, each found by a gate:
skip itself and lockfiles; special-case the double-branded names in BOTH scoped
and bare form; map path strings, not just paths; drop a level for the flattened
group; check disk rather than a stale index.

Rust build artifacts are NOT committed. An earlier draft of this commit swept
apps/desktop/src-tauri/target/debug in via `git add -A` — 3,859 files including
a 272MB .a, over GitHub's 100MB cap, so the push could never have landed. They
are gitignored now.

Co-Authored-By: Hanzo Dev <[email protected]>
2026-07-30 18:25:06 -07:00

384 lines
13 KiB
TypeScript

import { expect, test } from '@playwright/test'
import { getBoundingRect, setupPage } from './test-utils'
// helper: get opacity of element by id
async function getOpacity(page: any, id: string) {
return page.evaluate(
(sel: string) => parseFloat(getComputedStyle(document.getElementById(sel)!).opacity),
id
)
}
// Bug 1: delay should apply to both enter AND exit
test.describe('Popover hoverable delay', () => {
test.beforeEach(async ({ page }) => {
await setupPage(page, { name: 'PopoverHoverableDelayCase', type: 'useCase' })
await page.waitForLoadState('networkidle')
})
test('delay applies to enter: popover should not open before delay elapses', async ({
page,
}) => {
const trigger = page.locator('#delay-trigger')
const content = page.locator('#delay-content')
await expect(content).not.toBeVisible()
// hover over trigger
await trigger.hover()
// check immediately after hover (should NOT be open yet - delay is 400ms)
await expect(content).not.toBeVisible()
// wait less than delay - still should not open
await page.waitForTimeout(200)
await expect(content).not.toBeVisible()
// wait full delay time - should now be open
await page.waitForTimeout(300)
await expect(content).toBeVisible({ timeout: 3000 })
})
test('delay applies to exit: popover should not close before delay elapses', async ({
page,
}) => {
const trigger = page.locator('#delay-trigger')
const content = page.locator('#delay-content')
// hover to open (wait for delay)
await trigger.hover()
await page.waitForTimeout(800)
await expect(content).toBeVisible({ timeout: 3000 })
// move mouse far away (outside safe zone)
await page.mouse.move(10, 10)
// should still be visible right after (delay is 400ms)
await expect(content).toBeVisible()
// wait less than delay - still should be visible
await page.waitForTimeout(200)
await expect(content).toBeVisible()
// wait for delay to elapse + animation - should now be closing
await page.waitForTimeout(600)
await expect(content).not.toBeVisible({ timeout: 3000 })
})
})
// Bug 2: restMs should only apply to enter (not exit)
test.describe('Popover hoverable restMs', () => {
test.beforeEach(async ({ page }) => {
await setupPage(page, { name: 'PopoverHoverableRestMsCase', type: 'useCase' })
await page.waitForLoadState('networkidle')
})
test('restMs delays enter: popover should open after mouse rests', async ({ page }) => {
const trigger = page.locator('#restms-trigger')
const content = page.locator('#restms-content')
await expect(content).not.toBeVisible()
// hover over trigger
await trigger.hover()
// should not open immediately
await expect(content).not.toBeVisible()
// wait for restMs to elapse
await page.waitForTimeout(500)
await expect(content).toBeVisible({ timeout: 2000 })
})
test('exit without restMs: popover should close quickly after mouse leaves', async ({
page,
}) => {
const trigger = page.locator('#restms-trigger')
const content = page.locator('#restms-content')
// open it
await trigger.hover()
await page.waitForTimeout(500)
await expect(content).toBeVisible({ timeout: 2000 })
// move mouse away - with restMs but no delay, exit should be handled by safePolygon (quick)
await page.mouse.move(10, 10)
// exit should happen quickly (no restMs delay on exit, only animation duration ~200ms)
await expect(content).not.toBeVisible({ timeout: 1000 })
})
})
// Bug 3: exit animation should play when hoverable closes popover
test.describe('Popover hoverable exit animation', () => {
test.beforeEach(async ({ page }) => {
await setupPage(page, { name: 'PopoverHoverableExitAnimCase', type: 'useCase' })
await page.waitForLoadState('networkidle')
})
test('exit animation plays: content should still be visible briefly after mouse leaves', async ({
page,
}) => {
const trigger = page.locator('#exitanim-trigger')
const content = page.locator('#exitanim-content')
// hover to open
await trigger.hover()
await expect(content).toBeVisible({ timeout: 3000 })
// wait for enter animation to complete (500ms transition + buffer)
await page.waitForTimeout(700)
// confirm fully visible now
const opacityBefore = await content.evaluate((el) =>
parseFloat(getComputedStyle(el).opacity)
)
expect(opacityBefore).toBeGreaterThan(0.9)
// move mouse far away (outside safe zone) to trigger close
await page.mouse.move(10, 10)
// immediately after leaving - content should still be visible (animation in progress)
// the transition is 500ms so it should still be partially visible
await page.waitForTimeout(50)
const opacityDuring = await content.evaluate((el) =>
parseFloat(getComputedStyle(el).opacity)
)
// should be animating - opacity should be > 0 (not instantly gone to 0)
expect(opacityDuring).toBeGreaterThan(0)
// wait for animation to complete (500ms + buffer)
await page.waitForTimeout(600)
await expect(content).not.toBeVisible({ timeout: 2000 })
})
})
// Bug: safePolygon should allow hovering from trigger to content through the gap
test.describe('Popover hoverable safePolygon', () => {
test.beforeEach(async ({ page }) => {
await setupPage(page, { name: 'PopoverHoverableSafePolygonCase', type: 'useCase' })
await page.waitForLoadState('networkidle')
})
test('can hover from trigger to content through the gap', async ({ page }) => {
const trigger = page.locator('#safepoly-trigger')
const content = page.locator('#safepoly-content')
// hover trigger to open (restMs is 260ms)
await trigger.hover()
await page.waitForTimeout(350)
await expect(content).toBeVisible({ timeout: 200 })
// wait for enter animation to settle
await page.waitForTimeout(100)
// slowly move from trigger down to content (through the 80px offset gap)
const triggerBox = await trigger.boundingBox()
const contentBox = await content.boundingBox()
if (triggerBox && contentBox) {
const startY = triggerBox.y + triggerBox.height
const endY = contentBox.y + 10
const x = triggerBox.x + triggerBox.width / 2
for (let y = startY; y <= endY; y += 4) {
await page.mouse.move(x, y)
await page.waitForTimeout(10)
}
}
// wait well past any grace period or animation - if the popover closed
// during the gap crossing, it won't be visible after this
await page.waitForTimeout(500)
await expect(content).toBeVisible({ timeout: 1000 })
// verify content is fully opaque (not mid-exit-animation)
const opacity = await content.evaluate((el) =>
parseFloat(getComputedStyle(el).opacity)
)
expect(opacity).toBeGreaterThan(0.9)
})
test('restMs applies on re-hover (not just first hover)', async ({ page }) => {
const trigger = page.locator('#safepoly-trigger')
const content = page.locator('#safepoly-content')
// first hover: should respect restMs (260ms)
await trigger.hover()
// should NOT be open before restMs elapses
await page.waitForTimeout(100)
await expect(content).not.toBeVisible()
// wait for restMs
await page.waitForTimeout(250)
await expect(content).toBeVisible({ timeout: 2000 })
// move far away to close
await page.mouse.move(10, 10)
await page.waitForTimeout(500)
await expect(content).not.toBeVisible({ timeout: 2000 })
// re-hover: restMs should still apply (not open instantly)
await trigger.hover()
// check after 100ms - should NOT be open yet (restMs is 260ms)
await page.waitForTimeout(100)
await expect(content).not.toBeVisible()
// wait for restMs to elapse
await page.waitForTimeout(250)
await expect(content).toBeVisible({ timeout: 2000 })
})
})
// Bug: scoped multi-trigger hoverable - mimics WebsiteHeader.tsx pattern
// uses CSS driver since animatePosition needs a driver that supports classNames
test.describe('Popover hoverable scoped multi-trigger', () => {
test.beforeEach(async ({ page }) => {
await setupPage(page, {
name: 'PopoverHoverableScopedCase',
type: 'useCase',
searchParams: { animationDriver: 'css' },
})
await page.waitForLoadState('networkidle')
})
test('scoped: delay applies to enter - should not open immediately', async ({
page,
}) => {
const trigger = page.locator('#nav-trigger-about')
const content = page.locator('#nav-content')
await trigger.hover()
// should NOT be open right away (300ms delay)
await expect(content).not.toBeVisible()
// wait for delay
await page.waitForTimeout(450)
await expect(content).toBeVisible({ timeout: 2000 })
})
test('scoped: exit animation plays when hovering away', async ({ page }) => {
const trigger = page.locator('#nav-trigger-about')
const content = page.locator('#nav-content')
// open it
await trigger.hover()
await page.waitForTimeout(450)
await expect(content).toBeVisible({ timeout: 2000 })
// wait for enter animation to finish (500ms transition)
await page.waitForTimeout(700)
// verify content is correctly positioned below trigger (not stuck at y=0)
const triggerRect = await getBoundingRect(page, '#nav-trigger-about')
const contentRect = await getBoundingRect(page, '#nav-content')
expect(contentRect!.y).toBeGreaterThan(triggerRect!.y)
const opacityBefore = await getOpacity(page, 'nav-content')
expect(opacityBefore).toBeGreaterThan(0.9)
// move far away to trigger close
await page.mouse.move(10, 10)
// check opacity shortly after - should still be > 0 (animating out with 500ms transition)
await page.waitForTimeout(50)
const opacityDuring = await getOpacity(page, 'nav-content')
expect(opacityDuring).toBeGreaterThan(0)
// wait for animation to complete (500ms + buffer)
await page.waitForTimeout(600)
await expect(content).not.toBeVisible({ timeout: 2000 })
})
test('scoped: switching between triggers keeps popover open', async ({ page }) => {
const aboutTrigger = page.locator('#nav-trigger-about')
const blogTrigger = page.locator('#nav-trigger-blog')
const content = page.locator('#nav-content')
// open about
await aboutTrigger.hover()
await page.waitForTimeout(450)
await expect(content).toBeVisible({ timeout: 2000 })
// switch to blog trigger - popover should stay open
await blogTrigger.hover()
await page.waitForTimeout(100)
await expect(content).toBeVisible()
})
test('scoped: content → different trigger repositions popover', async ({ page }) => {
const aboutTrigger = page.locator('#nav-trigger-about')
const contactTrigger = page.locator('#nav-trigger-contact')
const content = page.locator('#nav-content')
// open at about
await aboutTrigger.hover()
await page.waitForTimeout(450)
await expect(content).toBeVisible({ timeout: 2000 })
await page.waitForTimeout(300)
const contentRectAtAbout = await getBoundingRect(page, '#nav-content')
// move into the popover content
await content.hover()
await page.waitForTimeout(100)
await expect(content).toBeVisible()
// move from content to "contact" trigger (rightmost)
await contactTrigger.hover()
await page.waitForTimeout(600)
// popover should still be visible and repositioned
await expect(content).toBeVisible()
const contentRectAtContact = await getBoundingRect(page, '#nav-content')
// contact is to the right of about, so content x should shift right
expect(contentRectAtContact!.x).toBeGreaterThan(contentRectAtAbout!.x + 20)
})
test('scoped: content → gap → different trigger repositions popover', async ({
page,
}) => {
const aboutTrigger = page.locator('#nav-trigger-about')
const contactTrigger = page.locator('#nav-trigger-contact')
const content = page.locator('#nav-content')
// open at about
await aboutTrigger.hover()
await page.waitForTimeout(450)
await expect(content).toBeVisible({ timeout: 2000 })
await page.waitForTimeout(300)
const contentRectAtAbout = await getBoundingRect(page, '#nav-content')
// move into the popover content
await content.hover()
await page.waitForTimeout(100)
await expect(content).toBeVisible()
// move mouse to empty space BELOW the content (not to a trigger)
// this simulates the recording where mouse exits content into the page gap
const contentBox = await content.boundingBox()
if (contentBox) {
await page.mouse.move(
contentBox.x + contentBox.width / 2,
contentBox.y + contentBox.height + 50
)
}
// wait a bit for the popover to close (safePolygon should eventually close it)
await page.waitForTimeout(300)
// now move to a different trigger (contact)
await contactTrigger.hover()
// wait for restMs (300ms) + buffer
await page.waitForTimeout(500)
// popover should be visible and repositioned at the new trigger
await expect(content).toBeVisible({ timeout: 2000 })
const contentRectAtContact = await getBoundingRect(page, '#nav-content')
// contact is to the right of about, so content x should shift right
expect(contentRectAtContact!.x).toBeGreaterThan(contentRectAtAbout!.x + 20)
})
})