🎨 Palette: 새로고침 버튼 로딩 상태 및 링크 접근성 개선

[seonghobae]

💡 What: 새로고침 버튼에 대한 로딩(disabled) 상태 추가 및 Open artifact 링크의 스크린 리더 접근성 향상
🎯 Why: 사용자가 데이터를 불러오는 중 중복 클릭을 방지하여 직관적인 피드백을 제공하고, 외부 창으로 열리는 링크의 용도를 스크린 리더 환경에 명확하게 전달하기 위함입니다.
📸 Before/After: (로컬 스크린샷 및 Playwright 동영상 첨부 완료)
♿ Accessibility: aria-label 및 키보드/시각적 :disabled 스타일링 향상

---

PR created automatically by Jules for task 12953156576872889856 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

Adds clearer loading/disabled UX for the “Refresh” button during preview polling, and improves accessibility signaling for the “Open artifact” link when it opens a new tab/window.

Changes:

• Disable the refresh button while loading/polling and re-enable it on error/success.
• Add target="_blank" plus an accessible label to the artifact link.
• Update button hover/disabled styling using native :disabled / :not(:disabled) selectors (and document the design constraint in .Jules/palette.md).

Reviewed changes

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

File	Description
src/main/resources/static/assets/viewer/viewer.js	Disables refresh during loading; adds new-tab behavior and screen-reader label for the artifact link.
src/main/resources/static/assets/viewer/viewer.css	Prevents hover styling on disabled buttons and adds a disabled visual style.
.Jules/palette.md	Documents the “no new custom CSS classes for UI state” approach for disabled states.

---

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

[opencode-agent]

OpenCode Review Overview

• Head SHA: 50424fc5a94dc3ef1b23ff6391e29241fe356c27
• Workflow run: 28302002520
• Workflow attempt: 1
• Gate result: REQUEST_CHANGES (approval step)

Pull request overview

OpenCode reviewed the current-head evidence but found unresolved human review threads before approval.

Findings

1. HIGH .github/workflows/opencode-review.yml:1 - Unresolved human review thread blocks automated approval

• Problem: OpenCode reached an APPROVE control result, but the approval step found unresolved, non-outdated human review thread evidence on the current pull request.
• Root cause: Human review feedback can arrive after bounded model evidence is prepared, so the approval step must re-query GitHub immediately before publishing an approval.
• Fix: Address or resolve the listed human review thread(s), then re-run OpenCode on the current head.
• Regression test: Keep the approval gate querying reviewThreads(first: 100) after model output and before create_pull_review APPROVE.

Review thread evidence

Latest unresolved human review thread evidence

src/main/resources/static/assets/viewer/viewer.js line 77

• Latest human comment: @copilot-pull-request-reviewer at 2026-06-27T21:16:50Z
• Comment URL: 🎨 Palette: 새로고침 버튼 로딩 상태 및 링크 접근성 개선 #38 (comment)
• Comment excerpt: For links opened with target="_blank", include "noreferrer" in rel to prevent leaking referrer data in browsers that don’t honor the page-level referrer policy consistently, and to follow standard security guidance for external-tab links.

.Jules/palette.md line 3

• Latest human comment: @copilot-pull-request-reviewer at 2026-06-27T21:16:50Z

• Comment URL: 🎨 Palette: 새로고침 버튼 로딩 상태 및 링크 접근성 개선 #38 (comment)

• Comment excerpt: In HTML, boolean attributes like "disabled" are enabled by presence; using disabled="true" in guidance can mislead readers into thinking disabled="false" would work. Recommend documenting the boolean form (disabled) instead.

• Result: REQUEST_CHANGES

• Reason: unresolved human review thread(s) were present before approval.

• Head SHA: 50424fc5a94dc3ef1b23ff6391e29241fe356c27

• Workflow run: 28302002520

• Workflow attempt: 1

Change Flow DAG

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

Loading

[opencode-agent]

Pull request overview

OpenCode reviewed the current-head evidence but found unresolved human review threads before approval.

Findings

1. HIGH .github/workflows/opencode-review.yml:1 - Unresolved human review thread blocks automated approval

• Problem: OpenCode reached an APPROVE control result, but the approval step found unresolved, non-outdated human review thread evidence on the current pull request.
• Root cause: Human review feedback can arrive after bounded model evidence is prepared, so the approval step must re-query GitHub immediately before publishing an approval.
• Fix: Address or resolve the listed human review thread(s), then re-run OpenCode on the current head.
• Regression test: Keep the approval gate querying reviewThreads(first: 100) after model output and before create_pull_review APPROVE.

Review thread evidence

Latest unresolved human review thread evidence

src/main/resources/static/assets/viewer/viewer.js line 77

• Latest human comment: @copilot-pull-request-reviewer at 2026-06-27T21:16:50Z
• Comment URL: #38 (comment)
• Comment excerpt: For links opened with target="_blank", include "noreferrer" in rel to prevent leaking referrer data in browsers that don’t honor the page-level referrer policy consistently, and to follow standard security guidance for external-tab links.

.Jules/palette.md line 3

• Latest human comment: @copilot-pull-request-reviewer at 2026-06-27T21:16:50Z

• Comment URL: #38 (comment)

• Comment excerpt: In HTML, boolean attributes like "disabled" are enabled by presence; using disabled="true" in guidance can mislead readers into thinking disabled="false" would work. Recommend documenting the boolean form (disabled) instead.

• Result: REQUEST_CHANGES

• Reason: unresolved human review thread(s) were present before approval.

• Head SHA: 50424fc5a94dc3ef1b23ff6391e29241fe356c27

• Workflow run: 28302002520

• Workflow attempt: 1

[seonghobae]

중복 정리: 이 PR은 'async 버튼 로딩 상태' 클러스터의 중복입니다. 정본으로 #71 (APPROVED + CI green + MERGEABLE)을 유지합니다. 동일 파일을 수정해 동시 병합이 불가능하므로 정리합니다. 고유하게 필요한 변경이 있으면 #71에 반영하거나 재오픈하세요.

[google-labs-jules]

중복 정리: 이 PR은 'async 버튼 로딩 상태' 클러스터의 중복입니다. 정본으로 #71 (APPROVED + CI green + MERGEABLE)을 유지합니다. 동일 파일을 수정해 동시 병합이 불가능하므로 정리합니다. 고유하게 필요한 변경이 있으면 #71에 반영하거나 재오픈하세요.

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