Skip to content

fix: show citation highlights after image load - #152

Merged
suguanYang merged 1 commit into
stagingfrom
fix/wangbinqi/citation-highlight-visibility
Aug 24, 2026
Merged

fix: show citation highlights after image load#152
suguanYang merged 1 commit into
stagingfrom
fix/wangbinqi/citation-highlight-visibility

Conversation

@suguanYang

Copy link
Copy Markdown
Contributor

Summary

  • fix citation highlight and pulse colors that were invalid when composed with the existing --ring color token
  • start region animations only after page or image assets finish loading, preserving the full temporary highlight duration on slow images
  • add component and headless Chromium coverage that blocks the image request and verifies the overlay mounts only after load

Test plan

  • pnpm exec vitest run src/components/parsed-chunk-card.test.ts (14 passed)
  • pnpm typecheck
  • pnpm exec eslint src/components/parsed-chunk-card.tsx src/components/parsed-chunk-card.test.ts e2e/citation-dedupe.e2e.ts
  • PLAYWRIGHT_EXTERNAL_WEB_SERVER=1 pnpm exec playwright test e2e/citation-dedupe.e2e.ts --project=chromium (3 passed)

Made with Cursor

Use valid token colors and delay region animation until page images are visible so slow assets do not consume the temporary highlight.

Co-authored-by: Cursor <cursoragent@cursor.com>
@vercel

vercel Bot commented Aug 24, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
knowhere-notebook-staging Ready Ready Preview Aug 24, 2026 2:04am
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
knowhere-notebook Ignored Ignored Aug 24, 2026 2:04am

Request Review

@suguanYang
suguanYang merged commit 673ab1f into staging Aug 24, 2026
7 checks passed
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