mirror of
https://github.com/hanzoai/gui.git
synced 2026-10-25 02:31:41 +00:00
zeekayandHanzo Dev
69c6e860d7
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]>
94 lines
3.1 KiB
TypeScript
94 lines
3.1 KiB
TypeScript
import { expect, test } from '@playwright/test'
|
|
import { setupPage } from './test-utils'
|
|
|
|
// Use smaller viewport to ensure menu needs to scroll
|
|
test.use({ viewport: { width: 800, height: 600 } })
|
|
|
|
test.describe('Menu Overflow', () => {
|
|
test.beforeEach(async ({ page }) => {
|
|
await setupPage(page, { name: 'MenuOverflowCase', type: 'useCase' })
|
|
})
|
|
|
|
test('menu content is constrained to max height', async ({ page }) => {
|
|
await page.waitForLoadState('networkidle')
|
|
|
|
const trigger = page.getByTestId('menu-trigger')
|
|
await trigger.click()
|
|
await page.waitForTimeout(300)
|
|
|
|
const menuContent = page.getByTestId('menu-content')
|
|
await expect(menuContent).toBeVisible()
|
|
|
|
// get menu content bounding box and viewport dimensions
|
|
const boundingBox = await menuContent.boundingBox()
|
|
const viewportSize = page.viewportSize()
|
|
|
|
expect(boundingBox).not.toBeNull()
|
|
expect(viewportSize).not.toBeNull()
|
|
|
|
if (boundingBox && viewportSize) {
|
|
// menu should not extend beyond viewport
|
|
expect(boundingBox.y).toBeGreaterThanOrEqual(0)
|
|
expect(boundingBox.y + boundingBox.height).toBeLessThanOrEqual(viewportSize.height)
|
|
// menu height should be constrained (less than total items would need)
|
|
expect(boundingBox.height).toBeLessThan(1000) // 30 items at ~33px each would be ~1000px
|
|
}
|
|
})
|
|
|
|
test('menu scroll view is scrollable', async ({ page }) => {
|
|
await page.waitForLoadState('networkidle')
|
|
|
|
const trigger = page.getByTestId('menu-trigger')
|
|
await trigger.click()
|
|
await page.waitForTimeout(300)
|
|
|
|
const menuScrollView = page.getByTestId('menu-scroll-view')
|
|
await expect(menuScrollView).toBeVisible()
|
|
|
|
// check that scroll view has overflow (is scrollable)
|
|
const isScrollable = await menuScrollView.evaluate((el) => {
|
|
return el.scrollHeight > el.clientHeight
|
|
})
|
|
|
|
expect(isScrollable).toBe(true)
|
|
})
|
|
|
|
test('scrollbars are hidden', async ({ page }) => {
|
|
await page.waitForLoadState('networkidle')
|
|
|
|
const trigger = page.getByTestId('menu-trigger')
|
|
await trigger.click()
|
|
await page.waitForTimeout(300)
|
|
|
|
const menuScrollView = page.getByTestId('menu-scroll-view')
|
|
await expect(menuScrollView).toBeVisible()
|
|
|
|
// check that scrollbar-width is none (Firefox/modern browsers)
|
|
const scrollbarWidth = await menuScrollView.evaluate((el) => {
|
|
return window.getComputedStyle(el).scrollbarWidth
|
|
})
|
|
|
|
expect(scrollbarWidth).toBe('none')
|
|
})
|
|
|
|
test('first item is visible and last item requires scrolling', async ({ page }) => {
|
|
await page.waitForLoadState('networkidle')
|
|
|
|
const trigger = page.getByTestId('menu-trigger')
|
|
await trigger.click()
|
|
await page.waitForTimeout(300)
|
|
|
|
const menuScrollView = page.getByTestId('menu-scroll-view')
|
|
await expect(menuScrollView).toBeVisible()
|
|
|
|
// first item should be visible
|
|
const firstItem = page.getByTestId('menu-item-1')
|
|
await expect(firstItem).toBeVisible()
|
|
|
|
// last item should exist but not be in view (requires scrolling)
|
|
const lastItem = page.getByTestId('menu-item-30')
|
|
await expect(lastItem).toBeAttached()
|
|
await expect(lastItem).not.toBeInViewport()
|
|
})
|
|
})
|