Skip to content

Add page navigation to PDF previews - #335

Merged
jeonghun-jj-lee merged 4 commits into
local/amicodefrom
334-add-page-navigation-to-pdf-previews
Sep 11, 2026
Merged

Add page navigation to PDF previews#335
jeonghun-jj-lee merged 4 commits into
local/amicodefrom
334-add-page-navigation-to-pdf-previews

Conversation

@jeonghun-jj-lee

@jeonghun-jj-lee jeonghun-jj-lee commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

Closes #334

Summary

  • add PDF page navigation before zoom
  • make the page and zoom pills share one translucent surface
  • support manual page entry with strict integer validation; invalid or out-of-range values restore the current page

Testing

  • bun run test:e2e -- preview-baseline-contract.spec.ts
  • bun run typecheck

@jeonghun-jj-lee jeonghun-jj-lee linked an issue Sep 11, 2026 that may be closed by this pull request
9 tasks
@coderabbitai

coderabbitai Bot commented Sep 11, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Advanced

Run ID: d7104004-cc0b-469c-83b5-ab03b9caab9d


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@jeonghun-jj-lee
jeonghun-jj-lee marked this pull request as ready for review September 11, 2026 01:55
@jeonghun-jj-lee
jeonghun-jj-lee merged commit a281458 into local/amicode Sep 11, 2026
4 checks passed
@jeonghun-jj-lee
jeonghun-jj-lee deleted the 334-add-page-navigation-to-pdf-previews branch September 11, 2026 02:24
@jeonghun-jj-lee
jeonghun-jj-lee restored the 334-add-page-navigation-to-pdf-previews branch September 11, 2026 02:36
@jeonghun-jj-lee
jeonghun-jj-lee deleted the 334-add-page-navigation-to-pdf-previews branch September 11, 2026 02:47
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.

Add page navigation to PDF previews

1 participant