fix: 수식 렌더링 개선 — TAC 높이 반영 + 한글 이탤릭 제거 (#174, #175)

[oksure]

요약

기존 PR #388 을 devel 기반으로 재작업한 PR 입니다. main 으로 직접 올린 점 죄송합니다.

1. TAC 높이에 HWP 권위값 사용 (#174)

• 인라인 수식 높이를 eq.common.height (HWP 저장 값) 기준으로 설정
• X/Y 스케일링 동시 적용으로 수식 bbox 정확도 향상
• 큰 수식이 인접 행과 겹치는 문제 해결

2. 한글(CJK) 이탤릭 제거 (#175)

• 수식 내 CJK 문자에 이탤릭 스타일링 및 너비 보정 비적용
• CASES 표현식에서 한글 행이 겹치는 문제 해결
• 회귀 테스트 추가 (CASES overlap / width)

변경 파일 (4개)

• src/renderer/svg.rs — 수식 SVG 배치 시 Y축 스케일링 추가
• src/renderer/layout/paragraph_layout.rs — HWP 수식 높이 우선 + 베이스라인 스케일 조정
• src/renderer/equation/svg_render.rs — CJK 수식 텍스트 이탤릭 제거
• src/renderer/equation/layout.rs — CJK 이탤릭 너비 보정 제외 + 회귀 테스트

테스트

• cargo test — 전체 통과
• cargo clippy -- -D warnings — 경고 0

Closes #174, #175

[Copilot]

Pull request overview

Improves equation rendering to better match HWP composition results by using stored equation height metrics and by preventing unintended italic styling/width adjustments for CJK text in equations.

Changes:

• Apply both X/Y scaling when placing equation SVG fragments to better match stored (HWP) equation bounding boxes.
• Prefer HWP-stored equation height (eq.common.height) for inline equation layout and proportionally adjust equation baseline placement.
• Detect CJK text in equations and avoid applying italic styling and italic width correction; adds regression tests for CASES overlap and CJK width behavior.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.

File	Description
src/renderer/svg.rs	Adds Y-axis scaling (in addition to X) when embedding equation SVG content into the page SVG.
src/renderer/layout/paragraph_layout.rs	Uses HWP-stored equation height for inline placement and scales baseline alignment when height differs from layout-derived height.
src/renderer/equation/svg_render.rs	Skips italic styling for CJK text when rendering equation SVG <text> elements.
src/renderer/equation/layout.rs	Skips italic width correction for CJK equation text and adds regression tests for CASES overlap + CJK width.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

test_cases_korean_no_overlap currently only checks for overlap inside nested if let matches; if the CASES layout shape changes (e.g., not Paren or body not a Row), the test will still pass without validating the no-overlap invariant. Consider making the expected structure explicit (or returning an error) so the test fails when it can’t reach the overlap assertion.

Suggested change

	if let LayoutKind::Paren { body, .. } = &lb.kind {
	if let LayoutKind::Row(rows) = &body.kind {
	assert!(rows.len() >= 2, "CASES should have at least 2 rows");
	let row1 = &rows[0];
	let row2 = &rows[1];
	let row1_bottom = row1.y + row1.height;
	let row2_top = row2.y;
	assert!(row2_top >= row1_bottom,
	"CASES rows should not overlap: row1 bottom={:.1}, row2 top={:.1}",
	row1_bottom, row2_top);
	}
	}
	let body = match &lb.kind {
	LayoutKind::Paren { body, .. } => body,
	other => panic!(
	"CASES should be laid out as Paren, got {:?}",
	other
	),
	};
	let rows = match &body.kind {
	LayoutKind::Row(rows) => rows,
	other => panic!(
	"CASES body should be laid out as Row, got {:?}",
	other
	),
	};
	assert!(rows.len() >= 2, "CASES should have at least 2 rows");
	let row1 = &rows[0];
	let row2 = &rows[1];
	let row1_bottom = row1.y + row1.height;
	let row2_top = row2.y;
	assert!(
	row2_top >= row1_bottom,
	"CASES rows should not overlap: row1 bottom={:.1}, row2 top={:.1}",
	row1_bottom,
	row2_top
	);

[Copilot]

This change removes italic styling for CJK in the SVG equation renderer, but the Canvas equation renderer still forces italic for LayoutKind::Text (see src/renderer/equation/canvas_render.rs:42-47). If Canvas output is user-facing (used by src/renderer/web_canvas.rs), the “CJK not italic” fix will be inconsistent across backends; consider applying the same CJK handling there as well.

[Copilot]

The inline-equation placement logic (HWP height priority + baseline scaling) is duplicated in two separate code paths. To reduce the risk of future divergence, consider extracting this into a small helper/closure that returns (eq_h, eq_y) given (layout_box, eq.common.height, baseline, y).

[oksure]

Copilot 리뷰 피드백 반영했습니다 (dbaa74b):

1. 테스트 명시적 실패 (layout.rs) — if let silent pass → match + panic!으로 변경. 실제로 전체 수식이 Row[subscript, =, Paren{cases}] 구조임을 확인하고 테스트가 올바르게 CASES Paren을 찾아 overlap 검증하도록 수정.

2. Canvas CJK 이탤릭 일관성 (canvas_render.rs) — SVG 렌더러와 동일한 CJK 범위 검사를 Canvas 렌더러에도 적용. LayoutKind::Text에서 CJK 포함 시 italic=false로 set_font 호출.

3. Helper 추출 (paragraph_layout.rs) — 인라인 수식 배치 로직 중복 해소 제안은 유효하나, 이 PR scope에서는 보류합니다. 향후 리팩토링 시 반영하겠습니다.

cargo test 전체 통과, cargo clippy -- -D warnings 경고 0.

[edwardkim]

완료. `devel` 에 cherry-pick (작성자 attribution 보존) + 메인테이너 후속 정정으로 머지됨 (commit `0d51736`).

이슈 #174, #175 의 SVG 경로 정정이 적용됐습니다.

머지 commit

작성자 attribution 보존:

• `2e62ef0` (← `5cf2b79`, @oksure) — 수식 렌더링 개선 (TAC 높이 + 한글 이탤릭)
• `9d4669e` (← `dbaa74b`, @oksure) — Copilot 리뷰 반영

메인테이너 후속 정정 (Canvas 경로 SVG 일치화):

• `7048d3e` — Canvas 분수선 y `- AXIS_HEIGHT` 정정
• `dff4b07` — web_canvas.rs Equation X/Y 스케일 적용
• `d47b7c7` — Canvas Limit `fi = fs` 정정

후속 정정 정황

작업지시자가 web 에디터 (Canvas 경로) 와 SVG 경로 시각 비교에서 다음 결함 발견:

• Canvas 의 분수선이 분모와 겹침
• Canvas 의 `lim` 글자가 비정상으로 큼
• Canvas 가 SVG 의 Y축 스케일링 정정을 적용 안 함 (수식이 bbox 와 다른 영역에 그려짐)

각 결함의 원인은 Canvas 경로가 SVG 경로와 일관되지 않은 코드 조각 (`font_size_from_box` 사용, scale 누락 등). 본 PR 후속 정정으로 SVG 경로와 일치화.

검증

• `cargo test --lib`: 1031 passed (1029 → +2 작성자 신규)
• `cargo test --test svg_snapshot`: 6/6
• `cargo clippy --lib -- -D warnings`: warning 0건
• `cargo check --target wasm32-unknown-unknown --lib`: 통과
• WASM 빌드: 4,106,811 bytes
• 작업지시자 시각 판정 통과 (web 에디터 + SVG 일치)

참고

• 동일 작성자 머지 PR: #395 (그림 밝기/대비)
• 본 PR 의 부수 발견 (Canvas 경로 일관성) 은 후속 정정으로 흡수됨

좋은 기여 감사합니다.