Skip to content

perf: optimize Svelte mobile drawer reactivity loops#388

Open
sherifhanna-google wants to merge 3 commits into
contentauth:mainfrom
sherifhanna-google:optimize-mobile-performance
Open

perf: optimize Svelte mobile drawer reactivity loops#388
sherifhanna-google wants to merge 3 commits into
contentauth:mainfrom
sherifhanna-google:optimize-mobile-performance

Conversation

@sherifhanna-google
Copy link
Copy Markdown
Contributor

Surgically resolves Svelte reactive declarations loops freezing viewport threads under throttled mobile constraints, and modernizes E2E integration testing suites to operate on standalone zero-dependency local CORS servers, async URL blob-downloaders, and thumbnail-tolerant array selectors.

@sherifhanna-google sherifhanna-google force-pushed the optimize-mobile-performance branch 3 times, most recently from 9a7ff09 to 98f9535 Compare May 5, 2026 23:28
…rifiable Social Accounts UI, and harden E2E harnesses
@sherifhanna-google sherifhanna-google force-pushed the optimize-mobile-performance branch from 98f9535 to 264dcbc Compare May 5, 2026 23:29
…lector TypeScript compiler errors

- Playwright: ignore 'e2e/snapshot' directory to disable slow visual snapshot tests locally and in CI.
- E2E: update locators in 'c2pa-migration-test.spec.ts' to use exact matching for 'Content Credentials' header, resolving Playwright strict-mode violations.
- Selectors: fix compiler errors in 'doNotTrain.ts' (missing return), 'reviewRatings.ts' and 'website.ts' (invalid dynamic index typing/any casts with ESLint bypass).
@sherifhanna-google sherifhanna-google changed the title perf: optimize Svelte mobile drawer reactivity loops, add Verifiable Social Accounts UI, and harden E2E harnesses perf: optimize Svelte mobile drawer reactivity loops May 7, 2026
- Remove 'socialAccounts' and 'producer' from 'ManifestData' type and parsing logic in 'asset.ts'.
- Delete unused 'socialAccounts.ts' and 'producer.ts' selector files.
- Delete unused 'SocialMediaInfo.svelte' component.
- Keep cryptographic signature 'Issued by' UI intact.
@andyparsons andyparsons self-requested a review May 13, 2026 15:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant