Skip to content

fix(pages): follow empty lines beside a cell object with text on both sides - #661

Merged
airmang merged 1 commit into
mainfrom
fix/empty-lines-beside-a-cell-object-on-both-sides
Nov 4, 2026
Merged

airmang merged 1 commit into
mainfrom
fix/empty-lines-beside-a-cell-object-on-both-sides

Conversation

@airmang

@airmang airmang commented Nov 3, 2026

Copy link
Copy Markdown
Owner

무엇을 고쳤나

표 칸 안에 양쪽으로 글 자리가 있는 어울림 그림이나 도형이 있으면, estimate_pages는 그 문서를 통째로 지원 밖(text on both sides of … wrapped square in a cell)으로 두었습니다.

  • 그 개체 곁에 빈 줄만 오면 이제 추정합니다. 한/글은 그런 빈 줄을 개체 곁에 같은 높이의 빈 두 조각으로 두고, 칸 높이는 한쪽에만 자리가 있을 때와 같습니다.
  • 곁에 글이 오면 지금처럼 지원 밖으로 둡니다. 사유 문구는 text on both sides of an object wrapped square in a cell로 바뀝니다.
  • 한/글이 저장한 줄 캐시가 있는 칸은 캐시대로 줄을 놓습니다. 이때 같은 높이에 놓인 둘째 조각을 별개의 줄로 세서 (갈라진 행이 다시 시작한 것으로 봄) 칸을 조각 하나마다 한 줄씩 높게 잡던 결함이 있었는데, 함께 고쳤습니다.

시험

pages_cell_object_on_both_sides_beside_empty_lines(5쪽)를 더했습니다.

  • 칸의 빈 첫 문단에 15000×8000 사각형을 두었습니다. 어울림, 양쪽 흐름, 문단 왼쪽에서 12000입니다.
  • 경우는 4가지입니다. 아래로 0·2000 × 겹침 허용 0·1.
  • 사각형 문단 뒤에 빈 문단 6개와 글 문단 2개가 이어집니다.
  • 한/글이 저장한 줄 캐시가 있는 판과 지운 판 모두 한/글과 같게 놓입니다. 고치기 전에는 두 판 모두 실패합니다.

🤖 Generated with Claude Code

… sides

In a table cell, Hancom sets an empty line reaching an object wrapped
square with room for text on both sides beside it, two empty pieces at
one height, the cell as tall as with room on one side. The estimate now
follows such cells instead of leaving them unsupported; text beside such
an object is still not followed. Reading a cell's layout caches, a
line's second piece at its first's height no longer counts as a line of
its own, which made the cell taller by a line for each.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

@airmang airmang left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

검토 head 96536b0 / base 5ca12c7. _cell_band/_clear_of의 양쪽 여유 표지 보존, 캐시 있는 두 조각의 같은 높이 처리, 기존 셀 줄 배치와 호출부를 검토했습니다. 정확한 head의 전체 CI 7,639 passed/18 skipped/1 xfailed 및 Ruff/mypy/Pyright 로그를 확인했고, 로컬 관련 53개를 head와 최신 main 통합 tree be55a4d21b7ea68f86ed4eb44ee261a0ce71856d에서 각각 통과했습니다. 추가 공개 API 변형으로 그림 옆 빈 문단에 글을 넣었을 때 캐시 없는 경우에는 both-sides 사유로 지원 불가, 저장된 유효 캐시가 있으면 그대로 배치됨도 확인했습니다. 이 변경에서 미해결 결함은 발견하지 못했습니다. 자기 PR이므로 COMMENT로 검토를 기록하며, 병합 직전에 head/base/CI/미해결 스레드를 다시 확인합니다.

@airmang
airmang merged commit edfaa65 into main Nov 4, 2026
13 checks passed
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