mirror of
https://github.com/Comfy-Org/ComfyUI_frontend.git
synced 2026-05-13 09:16:20 +00:00
*PR Created by the Glary-Bot Agent* --- ## Summary Replaces `right1.webp` with `right1.webm` in the VFX panel of `UseCaseSection`. `BlobMedia` already auto-detects `.webm` URLs and mounts a `<video>` element (with the `.webp` as poster), so this single URL swap is the only change required — matching the pattern used by the other 4 use-case panels. ## Files changed - `apps/website/src/components/home/UseCaseSection.vue` — swap `right1.webp` → `right1.webm`. ## Verification - `pnpm exec nx run website:typecheck` — clean - `pnpm exec eslint` on changed file — clean - `pnpm exec oxfmt --check` — clean - pre-commit lint-staged hooks — passed ## Reviewer note (Oracle finding) VFX is the default active panel, so the homepage's initial right-rail asset moves from ~131 KB `.webp` to ~4.1 MB `.webm`. Behaviorally consistent with the other 4 panels (which already use `.webm`), but worth confirming whether `right1.webm` should be re-encoded smaller on the CDN before promoting this PR out of draft. ┆Issue is synchronized with this [Notion page](https://www.notion.so/PR-12040-fix-use-webm-video-for-VFX-use-case-right-asset-3596d73d365081829976f37b733840f1) by [Unito](https://www.unito.io) --------- Co-authored-by: Glary-Bot <glary-bot@users.noreply.github.com> Co-authored-by: Amp <amp@ampcode.com>
58 lines
1.2 KiB
TypeScript
58 lines
1.2 KiB
TypeScript
import type { PlaywrightTestConfig } from '@playwright/test'
|
|
import { defineConfig, devices } from '@playwright/test'
|
|
|
|
const maybeLocalOptions: PlaywrightTestConfig = process.env.PLAYWRIGHT_LOCAL
|
|
? {
|
|
timeout: 30_000,
|
|
retries: 0,
|
|
workers: 1,
|
|
use: {
|
|
baseURL: 'http://localhost:4321',
|
|
trace: 'on',
|
|
video: 'on'
|
|
}
|
|
}
|
|
: {
|
|
retries: process.env.CI ? 2 : 0,
|
|
use: {
|
|
baseURL: 'http://localhost:4321',
|
|
trace: 'on-first-retry'
|
|
}
|
|
}
|
|
|
|
export default defineConfig({
|
|
testDir: './e2e',
|
|
fullyParallel: true,
|
|
forbidOnly: !!process.env.CI,
|
|
reporter: process.env.CI
|
|
? [['html'], ['json', { outputFile: 'results.json' }]]
|
|
: 'html',
|
|
expect: {
|
|
toHaveScreenshot: { maxDiffPixels: 100 }
|
|
},
|
|
...maybeLocalOptions,
|
|
webServer: {
|
|
command: 'pnpm preview',
|
|
port: 4321,
|
|
reuseExistingServer: !process.env.CI
|
|
},
|
|
projects: [
|
|
{
|
|
name: 'desktop',
|
|
use: { ...devices['Desktop Chrome'] },
|
|
grepInvert: /@mobile|@visual/
|
|
},
|
|
{
|
|
name: 'mobile',
|
|
use: { ...devices['Pixel 5'] },
|
|
grep: /@mobile/
|
|
},
|
|
{
|
|
name: 'visual',
|
|
use: { ...devices['Desktop Chrome'] },
|
|
grep: /@visual/,
|
|
fullyParallel: false
|
|
}
|
|
]
|
|
})
|