🎨 Palette: [UX improvement] 버튼 래퍼 ARIA 속성 및 아이콘 접근성 개선

[seonghobae]

💡 What:

• App.tsx와 Workspace.tsx에서 <button> 태그를 감싸는 툴팁용 <span> 래퍼 태그에 잘못 선언된 role="button"과 aria-disabled="true"를 제거했습니다.
• Workspace.tsx, RoleSwitcher.tsx, SectionRoadmap.tsx에서 텍스트와 함께 사용되는 시각적 장식용 Lucide 아이콘들에 aria-hidden="true" 속성을 추가했습니다.
• 접근성 수정 사항에 맞춰 App.test.tsx의 테스트 단언(assert)을 업데이트했습니다.

🎯 Why:

• ARIA 명세에 따르면 대화형 역할(Interactive roles, 예: button)은 내부에 다른 대화형 요소를 포함해서는 안 됩니다. 네이티브 <button>을 role="button"을 가진 <span>으로 감싸면 접근성 트리가 꼬이고 스크린 리더 사용자가 혼란을 겪을 수 있습니다.
• 텍스트 옆에 있는 꾸밈용 아이콘을 스크린 리더가 중복해서 읽는 것을 방지하여 청각적 노이즈를 줄이고 정보 전달의 명확성을 높였습니다.

📸 Before/After:
(시각적 변화는 없으며, DOM 트리의 접근성 속성 개선에 해당합니다.)

♿ Accessibility:

• 중첩된 대화형 역할 안티패턴을 제거하여 올바른 접근성 트리를 구성했습니다.
• 불필요한 시각 요소의 스크린 리더 노출을 차단하여 보이스오버(VoiceOver) 등 보조 기기 사용 시 더 깔끔하고 직관적인 내비게이션을 제공합니다.

---

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

[seonghobae]

Current-head security policy refresh added for this accessibility PR.

Verification run locally on the refreshed head:

• python3 scripts/checks/verify_supply_chain.py -> passed
• npm run check:security-notes -> passed
• git diff --cached --check before commit -> passed

This keeps #540 aligned with the shared RustSec/OSV policy already propagated across the open sale-readiness queue.

[google-labs-jules]

Current-head security policy refresh added for this accessibility PR.

Verification run locally on the refreshed head:

• python3 scripts/checks/verify_supply_chain.py -> passed
• npm run check:security-notes -> passed
• git diff --cached --check before commit -> passed

This keeps #540 aligned with the shared RustSec/OSV policy already propagated across the open sale-readiness queue.

Acknowledged.

[Copilot]

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

[seonghobae]

The branch tip moved after the first policy refresh, so I re-applied the same current-head security audit policy on top of the latest #540 head.

Verification run before this push:

• python3 scripts/checks/verify_supply_chain.py -> passed
• npm run check:security-notes -> passed
• staged diff whitespace check -> passed

[google-labs-jules]

The branch tip moved after the first policy refresh, so I re-applied the same current-head security audit policy on top of the latest #540 head.

Verification run before this push:

• python3 scripts/checks/verify_supply_chain.py -> passed
• npm run check:security-notes -> passed
• staged diff whitespace check -> passed

Acknowledged.

[seonghobae]

Closing as superseded by #541. The Palette branch repeatedly rewrote its head after security policy refresh commits and dropped the required RustSec/OSV policy files; #541 carries the same accessibility fix on a stable develop-based branch with the shared security policy refresh.

[google-labs-jules]

Closing as superseded by #541. The Palette branch repeatedly rewrote its head after security policy refresh commits and dropped the required RustSec/OSV policy files; #541 carries the same accessibility fix on a stable develop-based branch with the shared security policy refresh.

알겠습니다. 이 풀 리퀘스트가 #541에 의해 대체되어 폐기된 것을 확인했으며, 이 작업에 대한 추가 수정을 중단하겠습니다.

[opencode-agent]

OpenCode Review Overview

• Head SHA: da77322fa602f744c4dddbc377d2d5e1411136ab
• Workflow run: 28597359530
• Workflow attempt: 1
• Gate result: APPROVE (exit 0)

Changed-File Evidence Map

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

Loading