Skip to content

Task #1198: exam_social.hwp 중첩 표 셀 붙여넣기 경로 보존#1207

Merged
edwardkim merged 1 commit into
edwardkim:develfrom
postmelee:task-1198
Jun 1, 2026
Merged

Task #1198: exam_social.hwp 중첩 표 셀 붙여넣기 경로 보존#1207
edwardkim merged 1 commit into
edwardkim:develfrom
postmelee:task-1198

Conversation

@postmelee

Copy link
Copy Markdown
Collaborator

변경 요약

  • 중첩 표 셀에서 붙여넣기 대상이 바깥 셀로 해석되지 않도록 cellPath 기반 붙여넣기 native/WASM API를 추가했습니다.
  • rhwp-studio 붙여넣기 라우팅에서 cellPath.length > 1이면 내부 클립보드와 HTML 붙여넣기 모두 path 기반 API를 사용하도록 수정했습니다.
  • 붙여넣기 후 커서 위치가 같은 중첩 셀 경로를 유지하도록 cellParaIdx/cellPath 갱신을 보정했습니다.
  • samples/exam_social.hwp의 상단 성명 입력칸을 대상으로 내부/HTML 붙여넣기 회귀 테스트를 추가했습니다.

관련 이슈

Related: #1198

테스트

  • cargo fmt --check
  • cargo test --test issue_1198_nested_cell_paste -- --nocapture
  • cargo test --test issue_850_answer_sheet_name_hit_test issue_850_exam_social_answer_sheet_name_cell_keeps_outer_path -- --nocapture
  • cargo test --lib
  • wasm-pack build --target web --out-dir pkg
  • cd rhwp-studio && npm test
  • cd rhwp-studio && npm run build

메모

  • 기여자 규칙에 따라 이 PR은 이슈를 자동 종료하지 않습니다.
  • PR base는 devel입니다.

@edwardkim edwardkim self-requested a review June 1, 2026 07:36
@edwardkim edwardkim added the enhancement New feature or request label Jun 1, 2026
@edwardkim edwardkim added this to the v1.0.0 milestone Jun 1, 2026
edwardkim added a commit that referenced this pull request Jun 1, 2026
…#1198)

@postmelee. 1924 passed, opt-in 하위 호환, issue_1198 통합 + issue_850 회귀.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@edwardkim edwardkim merged commit e91385a into edwardkim:devel Jun 1, 2026
7 checks passed
@edwardkim

Copy link
Copy Markdown
Owner

머지했습니다(devel e91385ae). 감사합니다.

중첩 표 셀의 붙여넣기 경로 문제를 cellPath 기반 API 로 깔끔하게 풀어주셨습니다. 기존 단일 셀 API 를 보존하면서 path 변형을 추가하고 cellPath.length > 1 일 때만 라우팅하는 opt-in 구조라 일반 셀 경로에 회귀 위험이 없었고, 샘플/좌표에 의존하지 않고 cellPath 유무로 동작하는 점도 좋았습니다. copy source path API 대칭화를 후속으로 분리하신 범위 설정도 적절했습니다.

검증:

  • cargo test --tests 1924 passed (issue_1198 통합 2건 + issue_850 회귀 3건 포함)
  • rhwp-studio tsc && vite build 통과(신규 WASM API 타입 정합 포함), CI 전부 green
  • 신규 WASM API 노출을 위해 WASM 을 재빌드했습니다.

좋은 기여 감사합니다.

@edwardkim

Copy link
Copy Markdown
Owner

추가로, 메인테이너가 rhwp-studio 에서 exam_social.hwp성명 입력칸(중첩 표 안쪽 셀) 붙여넣기 동작을 직접 확인했습니다 — 붙여넣은 텍스트가 의도대로 안쪽 셀에 정확히 삽입됨을 검증(동작 테스트 통과)했습니다. 감사합니다.

edwardkim added a commit that referenced this pull request Jun 1, 2026
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
edwardkim added a commit that referenced this pull request Jun 1, 2026
@postmelee — 표 셀 텍스트 입력이 매 키마다 document-changed→refreshPages() full
refresh(visible page 전체 재렌더 + image retry reset)를 타던 비용을, 같은 cellPath
단일 insert/delete 만 page-local repaint 로 보내는 narrow invalidation 으로 축소.
선택안 A(document-page-invalidated 이벤트 분리) + C(image retry reset 생략).
보수적 화이트리스트 — 구조 변경/본문 입력은 기존 full refresh fallback.

Rust 무변경(rhwp-studio TS 전용). npm test 52 passed, Rust test --tests 1924 passed.
orders 충돌(우리 #1207 기록 + #1211 작업일지) 양쪽 보존 해소. 작업지시자 승인.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@postmelee postmelee deleted the task-1198 branch June 3, 2026 03:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants