fix(a11y): clarify table-edit modal close control

[seonghobae]

범위

테이블 편집 모달의 닫기 버튼 accessible name을 일반적인 닫기에서 현재 대화상자 맥락이 드러나는 테이블 편집 닫기로 좁혀 스크린 리더 사용자가 어떤 모달을 닫는지 구분할 수 있게 합니다. 동작, 저장/삭제 계약, 그래프 도메인 상태는 바꾸지 않습니다.

review → repair

CodeRabbit의 current-head pre-merge 검사에서 이 변경이 건드리는 production function의 docstring coverage가 0%로 보고됐습니다. exact descendant 10bf159de2963da0c16fd19d640fe1e5df852c8f에서 EditTableModal의 책임과 caller-owned graph mutation/focus boundary를 설명하는 JSDoc을 추가했습니다. 기존 라벨 regression은 그대로 유지합니다.

• protected base: main@8dc746920c12988f082e914879d95e13c9693535
• exact head: 10bf159de2963da0c16fd19d640fe1e5df852c8f
• compare: ahead 3 / behind 0, merge base가 protected main과 동일
• effective delta: EditTableModal.tsx, ModalCoverage.test.tsx 두 파일

증거 경계

현재 component test는 새 accessible name을 통해 닫기 동작을 찾는 정적/JSDOM 계약입니다. 실제 브라우저의 accessibility tree, 키보드 포커스 이동/복귀, 모바일·중간 너비 레이아웃 증거를 대신하지 않습니다. 이 repository의 현재 테이블 편집 surface는 한국어 하드코딩 UI이므로, 이 작은 접근성 수정만으로 KO/EN/JA/ZH/VI/ES/DE/FR translation-ledger 완결성을 주장하지 않습니다.

새 exact head에서 ci run 33851423830, SAST 33851423853, Security Scan 33851423896이 모두 queued입니다. predecessor 결과를 현재 head에 이전하지 않습니다.

Delivery Gate

• 의도성: PASS — accessible name을 대화상자 맥락에 한정했습니다.
• 기능 완전성: PARTIAL — component contract는 존재하지만 current-head browser/keyboard evidence가 없습니다.
• 콘텐츠 적합성: PASS — 새 장식 UI나 템플릿 문구를 추가하지 않습니다.
• 복원력: FAIL/PENDING — 실제 focus return, responsive/mobile 검증이 필요합니다.
• 증거성: FAIL/PENDING — exact-head hosted checks와 browser/a11y evidence가 아직 terminal이 아닙니다.
• 고유성: N/A — 시각 디자인을 바꾸지 않습니다.

Draft를 유지합니다. 정상 승격은 동일 exact head에서 required checks가 terminal GREEN이고, 실제 browser keyboard/accessibility evidence와 현재 review/thread 요구가 충족된 뒤에만 합니다. self-approval, bypass, no-op retrigger, force-push, gate weakening은 사용하지 않습니다.

Summary by CodeRabbit

• 접근성 개선
  • 테이블 편집 모달의 닫기 버튼 접근성 라벨을 더 구체적인 이름으로 변경했습니다.
  • 화면 낭독기 사용자가 현재 닫는 모달을 명확히 인식할 수 있습니다.

[google-labs-jules]

👋 Jules, reporting for duty! I'm here to lend a hand with this pull request.

When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down.

I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job!

For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with @jules. You can find this option in the Pull Request section of your global Jules UI settings. You can always switch back!

New to Jules? Learn more at jules.google/docs.

---

For security, I will only act on instructions from the user who triggered this task.

[coderabbitai]

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

• Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Team

Run ID: d93040c1-dcf0-4c36-9b1e-a0983e4ea73b

📥 Commits

Reviewing files that changed from the base of the PR and between 938627e and 10bf159.

📒 Files selected for processing (1)

• frontend/src/components/modals/EditTableModal.tsx

🚧 Files skipped from review as they are similar to previous changes (1)

• frontend/src/components/modals/EditTableModal.tsx

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.

---

📝 Walkthrough

Walkthrough

EditTableModal의 닫기 버튼 접근성 라벨을 닫기에서 테이블 편집 닫기로 변경했습니다. 컴포넌트 설명용 JSDoc과 관련 테스트도 갱신했습니다.

Changes

편집 모달 닫기 라벨

Layer / File(s)	Summary
닫기 라벨 및 테스트 갱신 frontend/src/components/modals/EditTableModal.tsx, frontend/src/components/modals/ModalCoverage.test.tsx	EditTableModal의 역할과 포커스 계약을 설명하는 JSDoc을 추가했습니다. 닫기 버튼의 aria-label을 테이블 편집 닫기로 변경했습니다. 테스트가 새 접근 가능한 이름을 사용하도록 수정했습니다.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Merge Risk: ⚪ Minimal · up to 10bf1

The modal close button now provides a clearer accessible label, with no identified merge-readiness risk.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 1 functions across 2 files.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	PR 제목은 테이블 편집 모달 닫기 컨트롤의 접근성 레이블을 명확하게 변경하는 주요 내용을 정확하고 간결하게 설명합니다.

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch palette-edit-table-modal-aria-label-1723042806679039715

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}