mirror of
https://github.com/hanzoai/gui.git
synced 2026-10-25 02:31:41 +00:00
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]>
96 lines
3.3 KiB
TypeScript
96 lines
3.3 KiB
TypeScript
import { expect, test } from '@playwright/test'
|
|
import { setupPage } from './test-utils'
|
|
import { TEST_IDS } from '../src/constants/test-ids'
|
|
|
|
/**
|
|
* Tests for themeShallow prop DOM structure stability
|
|
*
|
|
* The themeShallow prop should NOT affect DOM structure (span wrapping).
|
|
* It only affects whether grandchildren inherit the theme change.
|
|
*
|
|
* This test verifies that themeShallow doesn't cause unexpected DOM changes.
|
|
*/
|
|
|
|
test.beforeEach(async ({ page }) => {
|
|
await setupPage(page, { name: 'ThemeShallowCase', type: 'useCase' })
|
|
})
|
|
|
|
test('themeShallow does not add extra wrapper elements compared to normal theme', async ({
|
|
page,
|
|
}) => {
|
|
const normalContainer = page.locator(`#${TEST_IDS.themeShallowNormal}`)
|
|
const shallowContainer = page.locator(`#${TEST_IDS.themeShallowEnabled}`)
|
|
|
|
await expect(normalContainer).toBeVisible()
|
|
await expect(shallowContainer).toBeVisible()
|
|
|
|
// Get the DOM structure depth for both containers
|
|
// Count the number of wrapper elements between the container and its text content
|
|
const normalDepth = await normalContainer.evaluate((el) => {
|
|
let depth = 0
|
|
let current = el.querySelector('[class*="is_Theme"]')
|
|
while (current && current !== el) {
|
|
depth++
|
|
current = current.parentElement
|
|
}
|
|
return depth
|
|
})
|
|
|
|
const shallowDepth = await shallowContainer.evaluate((el) => {
|
|
let depth = 0
|
|
let current = el.querySelector('[class*="is_Theme"]')
|
|
while (current && current !== el) {
|
|
depth++
|
|
current = current.parentElement
|
|
}
|
|
return depth
|
|
})
|
|
|
|
// The DOM structure depth should be the same regardless of themeShallow
|
|
// themeShallow only affects theme context propagation, not DOM structure
|
|
expect(shallowDepth).toBe(normalDepth)
|
|
})
|
|
|
|
test('themeShallow container has consistent DOM structure', async ({ page }) => {
|
|
const container = page.locator(`#${TEST_IDS.themeShallowContainer}`)
|
|
const inner = page.locator(`#${TEST_IDS.themeShallowInner}`)
|
|
|
|
await expect(container).toBeVisible()
|
|
await expect(inner).toBeVisible()
|
|
|
|
// Snapshot the inner HTML structure to catch any regressions
|
|
const innerHtml = await inner.evaluate((el) => {
|
|
// Normalize the HTML by removing dynamic attributes like data-* IDs
|
|
const clone = el.cloneNode(true) as HTMLElement
|
|
clone.querySelectorAll('*').forEach((node) => {
|
|
// Remove dynamic attributes that change between runs
|
|
Array.from(node.attributes).forEach((attr) => {
|
|
if (
|
|
attr.name.startsWith('data-') &&
|
|
!attr.name.startsWith('data-test') &&
|
|
!attr.name.startsWith('data-disable')
|
|
) {
|
|
node.removeAttribute(attr.name)
|
|
}
|
|
})
|
|
})
|
|
return clone.innerHTML
|
|
})
|
|
|
|
// The inner container should have text content wrapped appropriately
|
|
expect(innerHtml).toContain('Inner content')
|
|
|
|
// Verify Theme wrapper spans are present (theme always wraps in span on web)
|
|
const themeSpans = await container.locator('span.is_Theme').count()
|
|
expect(themeSpans).toBeGreaterThan(0)
|
|
})
|
|
|
|
test('themeShallow prop does not change element tag type', async ({ page }) => {
|
|
const shallowContainer = page.locator(`#${TEST_IDS.themeShallowEnabled}`)
|
|
await expect(shallowContainer).toBeVisible()
|
|
|
|
// The container itself should be a div (Stack renders as div)
|
|
const tagName = await shallowContainer.evaluate((el) => el.tagName.toLowerCase())
|
|
expect(tagName).toBe('div')
|
|
})
|