Skip to content

Fix clickable links in PDF viewer#28

Merged
jdmcleod merged 3 commits intomainfrom
allow-clicking-links
Mar 11, 2026
Merged

Fix clickable links in PDF viewer#28
jdmcleod merged 3 commits intomainfrom
allow-clicking-links

Conversation

@jdmcleod
Copy link
Collaborator

Summary

  • add a regression test that verifies PDF link annotations render as clickable anchors
  • render annotation links in rm-pdf-page using PDF annotation data and viewport coordinates
  • layer annotation links above text/canvas so links are actually clickable

Verification

  • yarn test --run test/components/pdf-viewer.test.js (red before fix, green after)
  • yarn test --run
  • yarn build

Render PDF link annotations in rm-pdf-page so links are clickable, and add a regression test for external links.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@jdmcleod jdmcleod merged commit 4a5e944 into main Mar 11, 2026
1 check passed
@jdmcleod jdmcleod deleted the allow-clicking-links branch March 11, 2026 21:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants