Skip to content

[react-pdf] text layer와 canvas 위치 불일치 수정#221

Open
2-one-week wants to merge 2 commits into
mainfrom
fix/text-layer-canvas-position-mismatch
Open

[react-pdf] text layer와 canvas 위치 불일치 수정#221
2-one-week wants to merge 2 commits into
mainfrom
fix/text-layer-canvas-position-mismatch

Conversation

@2-one-week
Copy link
Copy Markdown
Member

@2-one-week 2-one-week commented Apr 16, 2026

  • CSS: line-height: 1, text-size-adjust: none 추가 (pdfjs-dist 공식 구현 기준)
  • CSS: transform-origin을 0% 0%로 변경
  • merge: text item 간 gap을 width에 포함하여 visual span 정확도 개선
  • tokenize: 매직 넘버(+5, +3.5, 4.5) 제거, 비례 분할 방식으로 변경

Related Issue

Describe your changes

Request

- CSS: line-height: 1, text-size-adjust: none 추가 (pdfjs-dist 공식 구현 기준)
- CSS: transform-origin을 0% 0%로 변경
- merge: text item 간 gap을 width에 포함하여 visual span 정확도 개선
- tokenize: 매직 넘버(+5, +3.5, 4.5) 제거, 비례 분할 방식으로 변경

Closes common-fe/pie/discussions/3483

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@2-one-week 2-one-week requested a review from a team as a code owner April 16, 2026 10:47
Copy link
Copy Markdown

@claude claude Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Code review skipped — your organization's overage spend limit has been reached.

Code review is billed via overage credits. To resume reviews, an organization admin can raise the monthly limit at claude.ai/admin-settings/claude-code.

Once credits are available, push a new commit or reopen this pull request to trigger a review.

@npayfebot
Copy link
Copy Markdown
Collaborator

npayfebot commented Apr 16, 2026

✅ Changeset detected

Latest commit: 08036d1

@naverpay/react-pdf package have detected changes.

If no version change is needed, please add skip-detect-change to the label.

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@naverpay/react-pdf 🐛 Patch
powered by: naverpay changeset detect-add actions

Fixes the mismatch between the text layer and canvas in react-pdf.
@2-one-week
Copy link
Copy Markdown
Member Author

/canary-publish

@npayfebot
Copy link
Copy Markdown
Collaborator

npayfebot commented Apr 16, 2026

NPM Packages

📦 @naverpay/react-pdf

Total Sizes: 20.9 kB

Total Changes: +17 B (0%)

ℹ️ View Unchanged Files
File Status Previous Size Updated Size Changed
/dist/esm/index.mjs - 310 B 310 B -
/dist/esm/utils/text.mjs 🛠️ 567 B 576 B +9 B (+2%)
/dist/esm/utils/pdf.mjs - 1.21 kB 1.21 kB -
/dist/esm/utils/link-service.mjs - 843 B 843 B -
/dist/esm/utils/debounce.mjs - 148 B 148 B -
/dist/esm/hooks/usePdfViewerPageWidth.mjs - 439 B 439 B -
/dist/esm/hooks/useIsomorphicLayoutEffect.mjs - 136 B 136 B -
/dist/esm/hooks/useInfiniteScroll.mjs - 303 B 303 B -
/dist/esm/contexts/pdf.mjs - 308 B 308 B -
/dist/esm/contexts/page.mjs - 447 B 447 B -
/dist/esm/components/PdfViewer.module.mjs - 116 B 116 B -
/dist/esm/components/PdfViewer.mjs - 1.25 kB 1.25 kB -
/dist/esm/components/Pages.mjs - 902 B 902 B -
/dist/esm/components/page/Canvas.mjs - 579 B 579 B -
/dist/esm/components/layer/Text.module.mjs - 108 B 108 B -
/dist/esm/components/layer/Text.mjs - 1.25 kB 1.25 kB -
/dist/esm/components/layer/Annotation.module.mjs - 106 B 106 B -
/dist/esm/components/layer/Annotation.mjs - 896 B 896 B -
/dist/cjs/index.js - 406 B 406 B -
/dist/cjs/utils/text.js 🛠️ 630 B 639 B +9 B (+1%)
/dist/cjs/utils/pdf.js - 1.28 kB 1.28 kB -
/dist/cjs/utils/link-service.js - 845 B 845 B -
/dist/cjs/utils/debounce.js - 211 B 211 B -
/dist/cjs/hooks/usePdfViewerPageWidth.js - 440 B 440 B -
/dist/cjs/hooks/useIsomorphicLayoutEffect.js - 192 B 192 B -
/dist/cjs/hooks/useInfiniteScroll.js - 300 B 300 B -
/dist/cjs/contexts/pdf.js - 375 B 375 B -
/dist/cjs/contexts/page.js - 516 B 516 B -
/dist/cjs/components/PdfViewer.module.js - 206 B 206 B -
/dist/cjs/components/PdfViewer.js - 1.31 kB 1.31 kB -
/dist/cjs/components/Pages.js - 978 B 978 B -
/dist/cjs/components/page/Canvas.js - 645 B 645 B -
/dist/cjs/components/layer/Text.module.js 🛠️ 118 B 117 B -1 B (-1%)
/dist/cjs/components/layer/Text.js - 1.33 kB 1.33 kB -
/dist/cjs/components/layer/Annotation.module.js - 189 B 189 B -
/dist/cjs/components/layer/Annotation.js - 969 B 969 B -

powered by: naverpay size-action

@npayfebot
Copy link
Copy Markdown
Collaborator

Published Canary Packages

@naverpay/react-pdf@1.1.2-canary.260416-08036d1

Copy link
Copy Markdown
Collaborator

@ttppggnnss ttppggnnss left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

감사합니다 🙇‍♂️

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.

3 participants