🎨 Palette: 간트 차트 닫기 버튼에 aria-keyshortcuts 추가

[seonghobae]

💡 What:

index.html의 간트 차트 모달 닫기 버튼(#close-gantt)에 aria-keyshortcuts="Escape" 속성을 추가했습니다.

🎯 Why:

기존에는 버튼의 title 속성에 "닫기 (Esc)"라고만 적혀 있어서, 마우스를 올릴 때 시각적인 힌트는 제공되었으나 스크린 리더와 같은 보조 기술을 사용하는 사용자에게는 키보드 단축키 정보가 명시적으로 전달되지 않는 문제가 있었습니다. 이를 개선하여 보조 기술 사용자가 Esc 키를 통해 모달을 닫을 수 있음을 인지할 수 있도록 명확한 힌트를 제공하기 위함입니다.

📸 Before/After:

Before:

<button id="close-gantt" type="button" class="icon-button close-button" aria-label="간트 차트 닫기" title="닫기 (Esc)"><span aria-hidden="true">✕</span></button>

After:

<button id="close-gantt" type="button" class="icon-button close-button" aria-label="간트 차트 닫기" title="닫기 (Esc)" aria-keyshortcuts="Escape"><span aria-hidden="true">✕</span></button>

♿ Accessibility:

• 스크린 리더가 버튼의 aria-keyshortcuts 속성을 읽어 사용자에게 Esc 키 단축키가 있음을 명확하게 알려줍니다.
• 기존 title에만 의존하던 키보드 단축키 힌트를 보조 기술 표준에 맞게 노출하여 접근성을 크게 향상시켰습니다.

---

PR created automatically by Jules for task 15846525753200385319 started by @seonghobae

[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.

[Copilot]

Pull request overview

This PR improves accessibility for the Gantt chart modal by explicitly exposing the Escape-key shortcut to assistive technologies via aria-keyshortcuts on the close button.

Changes:

• Add aria-keyshortcuts="Escape" to the Gantt modal close button (#close-gantt) in index.html.

---

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

[github-actions]

Pull request overview

OpenCode exhausted the configured model pool without a usable current-head review conclusion. This is not approval evidence, so the PR is blocked until a source-backed review can establish approval sufficiency or identify concrete fixes.

Findings

1. HIGH index.html:1 - OpenCode could not establish approval sufficiency

• Problem: every configured model path failed to produce a usable current-head control block.
• Root cause: model execution, timeout, export, normalization, or approval-gate validation did not complete after exponential retry across the configured model pool.
• Impact: approving from deterministic check state alone would miss PR-intent mismatches, missing files, edge-case bugs, robustness gaps, UX/DX regressions, security issues, and CodeGraph-backed base/head flow changes.
• Fix: rerun OpenCode after model availability recovers, or update the PR with the missing files, tests, docs, generated artifacts, and verification evidence needed for a source-backed review conclusion.
• Regression test: keep the approval gate posting REQUEST_CHANGES, not APPROVE or check-only failure, when no model produces a valid current-head review.

Summary

• Result: REQUEST_CHANGES
• Reason: coverage-evidence passed and peer GitHub Checks completed without failures, but no model produced a valid review control block.
• Deterministic evidence checked but not used for approval: current-head changed-file evidence (index.html); coverage-evidence result success; peer checks from statusCheckRollup excluding this OpenCode check.
• Model outcome: model_pool=exhausted; selected_model=none.
• Head SHA: 42608987608cca36fedf1e20dc1a57f8d2482c3d
• Workflow run: 28527193590
• Workflow attempt: 1

No PR approval was posted because model-output failure is not evidence that the PR has no blockers.

Changed-File Evidence Map

flowchart LR
  PR["PR changed files"] --> Evidence["OpenCode bounded evidence"]
  Evidence --> S1["Changed file: index.html"]
  S1 --> I1["repository behavior"]
  I1 --> R1["Review risk: Changed file: index.html"]
  R1 --> V1["required checks"]

Loading

[github-actions]

HIGH OpenCode could not establish approval sufficiency

• Problem: the model pool exhausted without a valid current-head review control block, so this changed line cannot be approved from deterministic check state alone.
• Impact: PR-intent mismatches, missing files, robustness bugs, UX/DX regressions, and CodeGraph-backed flow changes could be missed.
• Fix: rerun OpenCode after model availability recovers, or add the missing source/test/docs/generated verification evidence needed for a source-backed approval.
• Verification: rerun the OpenCode Review workflow and confirm it emits APPROVE or source-backed REQUEST_CHANGES for this head SHA.

[github-actions]

OpenCode Review Overview

• Head SHA: 42608987608cca36fedf1e20dc1a57f8d2482c3d
• Workflow run: 28527193590
• Workflow attempt: 1
• Gate result: REQUEST_CHANGES (approval step)

Pull request overview

OpenCode exhausted the configured model pool without a usable current-head review conclusion. This is not approval evidence, so the PR is blocked until a source-backed review can establish approval sufficiency or identify concrete fixes.

Findings

1. HIGH index.html:1 - OpenCode could not establish approval sufficiency

• Problem: every configured model path failed to produce a usable current-head control block.
• Root cause: model execution, timeout, export, normalization, or approval-gate validation did not complete after exponential retry across the configured model pool.
• Impact: approving from deterministic check state alone would miss PR-intent mismatches, missing files, edge-case bugs, robustness gaps, UX/DX regressions, security issues, and CodeGraph-backed base/head flow changes.
• Fix: rerun OpenCode after model availability recovers, or update the PR with the missing files, tests, docs, generated artifacts, and verification evidence needed for a source-backed review conclusion.
• Regression test: keep the approval gate posting REQUEST_CHANGES, not APPROVE or check-only failure, when no model produces a valid current-head review.

Summary

• Result: REQUEST_CHANGES
• Reason: coverage-evidence passed and peer GitHub Checks completed without failures, but no model produced a valid review control block.
• Deterministic evidence checked but not used for approval: current-head changed-file evidence (index.html); coverage-evidence result success; peer checks from statusCheckRollup excluding this OpenCode check.
• Model outcome: model_pool=exhausted; selected_model=none.
• Head SHA: 42608987608cca36fedf1e20dc1a57f8d2482c3d
• Workflow run: 28527193590
• Workflow attempt: 1

No PR approval was posted because model-output failure is not evidence that the PR has no blockers.

Changed-File Evidence Map

flowchart LR
  PR["PR changed files"] --> Evidence["OpenCode bounded evidence"]
  Evidence --> S1["Changed file: index.html"]
  S1 --> I1["repository behavior"]
  I1 --> R1["Review risk: Changed file: index.html"]
  R1 --> V1["required checks"]

Loading

[seonghobae]

Closing during org-wide PR backlog reduction. This looks like a low-priority automated maintenance PR (palette ux/a11y), while the repository has a large backlog of security, conflict, and required-workflow work. Reopen or recreate a fresh PR if this change is still needed.

[google-labs-jules]

Closing during org-wide PR backlog reduction. This looks like a low-priority automated maintenance PR (palette ux/a11y), while the repository has a large backlog of security, conflict, and required-workflow work. Reopen or recreate a fresh PR if this change is still needed.

Understood. Acknowledging that this work is now obsolete and stopping work on this task.