refactor: pdf viewer components

[fundon]

No description provided.

[graphite-app]

Your org has enabled the Graphite merge queue for merging into canary

Add the label “merge” to the PR and Graphite will automatically add it to the merge queue when it’s ready to merge. Or use the label “hotfix” to add to the merge queue as a hot fix.

You must have a Graphite account and log in to Graphite in order to use the merge queue. Sign up using this link.

[fundon]

• refactor: pdf viewer components #8784 👈 (View in Graphite)
• refactor(core): patch pdf viewer #8782
• feat(core): pdf preview #8569 : 5 other dependent PRs (#8615 , #8671 , #8720 and 2 others)
• feat: allow peek view to be closed by the caller && bump bs #8542
• canary

This stack of pull requests is managed by Graphite. Learn more about stacking.

[nx-cloud]

☁️ Nx Cloud Report

CI is running/has finished running commands for commit e2c5390. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this CI Pipeline Execution

---

✅ Successfully ran 1 target

• nx test:coverage @affine/monorepo

---

Sent with 💌 from NxCloud.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 70.25%. Comparing base (fbfc9e8) to head (e2c5390).
Report is 1 commits behind head on 61/patch-pdf.

Additional details and impacted files

@@              Coverage Diff              @@
##           61/patch-pdf    #8784   +/-   ##
=============================================
  Coverage         70.25%   70.25%           
=============================================
  Files               549      549           
  Lines             33898    33898           
  Branches           3043     3043           
=============================================
  Hits              23814    23814           
  Misses             9713     9713           
  Partials            371      371           

Flag	Coverage Δ
server-test	77.18% <ø> (ø)
unittest	45.87% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[graphite-app]

This console.log statement appears to be a debugging artifact and should be removed before merging to production. While logging is valuable during development, keeping debug logs in production code can impact performance and pollute logs with unnecessary information.

Spotted by Graphite Reviewer

Is this helpful? React 👍 or 👎 to let us know.