Skip to content

[-]: highLightPadding 반영 및 overlay 위치 보정#7

Merged
sjsjsj1246 merged 1 commit into
mainfrom
codex/add-highlight-padding-and-overlay-click-behavior
Mar 24, 2026
Merged

[-]: highLightPadding 반영 및 overlay 위치 보정#7
sjsjsj1246 merged 1 commit into
mainfrom
codex/add-highlight-padding-and-overlay-click-behavior

Conversation

@sjsjsj1246
Copy link
Copy Markdown
Owner

변경 사항

  • highLightPadding 기본값을 8px로 두고 highlight rect 계산에 실제 반영했습니다.
  • highlight frame을 box-sizing: border-box 기준으로 정리하고 padding에 맞춰 border radius를 조정했습니다.
  • info box가 viewport 경계 밖으로 나가지 않도록 좌우/상하 위치를 보정했습니다.
  • highlight padding 및 info box 위치 보정 회귀 테스트를 추가하고 README/docs 설명을 현재 동작에 맞게 업데이트했습니다.

테스트 결과

  • pnpm -C packages/main test 통과
  • pnpm build 통과
  • pnpm -F @react-tutorial-overlay/document build 통과

남은 리스크

  • overlay 좌표계는 기존처럼 window.scrollX/Y를 사용하므로, 깊은 스크롤 컨테이너 환경에 대한 별도 보강은 이번 범위에 포함하지 않았습니다.
  • 계획 문서 상태 업데이트는 merge 시점에 반영하는 게 맞아서 이번 PR에는 포함하지 않았습니다.

@vercel
Copy link
Copy Markdown

vercel Bot commented Mar 24, 2026

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

Project Deployment Actions Updated (UTC)
react-tutorial-overlay Ready Ready Preview, Comment Mar 24, 2026 2:38pm

@github-actions
Copy link
Copy Markdown
Contributor

packages/main coverage

Metric Result
Lines 95.89% (187/195)
Statements 95.52% (192/201)
Functions 91.66% (33/36)
Branches 83.89% (125/149)

Workflow run

@sjsjsj1246 sjsjsj1246 self-assigned this Mar 24, 2026
@sjsjsj1246 sjsjsj1246 merged commit 0e5eeb9 into main Mar 24, 2026
3 checks passed
@sjsjsj1246 sjsjsj1246 deleted the codex/add-highlight-padding-and-overlay-click-behavior branch March 24, 2026 14:45
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