🎨 Palette: ContextSection 의 불필요한 Chevron 스크린리더 출력 제거 by seonghobae · Pull Request #297 · ContextualWisdomLab/argos

seonghobae · 2026-07-22T20:31:27Z

💡 What: ContextSection 컴포넌트의 토글 버튼에 있는 ChevronUp/ChevronDown 아이콘에 `aria-hidden="true"` 속성을 추가했습니다.

🎯 Why: 이미 부모 `button` 요소가 `aria-expanded`를 통해 상태를 알려주고 있으므로, 불필요한 장식용 아이콘까지 스크린리더가 읽어버리는 문제를 방지하기 위함입니다. 이를 통해 시각 장애인 사용자가 겪는 인지 부하를 줄일 수 있습니다.

📸 Before/After: 시각적 변화 없음 (HTML Semantic 속성만 변경)

♿ Accessibility: 장식용/중복 정보를 제공하는 아이콘을 보조 기술에서 숨겨, 보다 깔끔한 스크린 리더 경험을 제공합니다.

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

Added aria-hidden="true" to the ChevronUp and ChevronDown icons in the ContextSection component to prevent screen readers from reading them out loud, as the state is already conveyed by aria-expanded on the parent button.

google-labs-jules · 2026-07-22T20:31:28Z

👋 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

ContextSection(대시보드 리포트 UI)의 토글 버튼에서 장식용 Chevron 아이콘이 스크린리더에 중복으로 읽히지 않도록 접근성 속성을 보완하는 PR입니다.

Changes:

토글 상태 아이콘(ChevronUp/ChevronDown)에 aria-hidden="true"를 추가해 보조기술에서 장식 요소를 숨김
(부수적으로) 파일 전반의 문자열/세미콜론/포맷팅이 함께 변경됨

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

+"use client";

-import React, { useState, useId, type ReactNode } from 'react'
-import { ChevronDown, ChevronUp } from 'lucide-react'
-import { cn } from '@/lib/utils'
+import React, { useState, useId, type ReactNode } from "react";
+import { ChevronDown, ChevronUp } from "lucide-react";
+import { cn } from "@/lib/utils";


        aria-expanded={open}
        aria-controls={contentId}
      >
        <h2 className="text-base font-medium">{title}</h2>


Updated multiple dependencies in `package.json` overrides to address high and moderate severity vulnerabilities detected by osv-scanner. Includes updates for `hono`, `js-yaml`, `sharp`, `@hono/node-server`, `fast-uri`, `brace-expansion`, and `body-parser`.

Copilot

Pull request overview

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

Files not reviewed (1)

pnpm-lock.yaml: Generated file

Comments suppressed due to low confidence (1)

packages/web/src/components/dashboard/reports/context-section.tsx:5

이 파일만 "use client"/import 구문이 쌍따옴표+세미콜론 스타일로 변경되어, 같은 디렉터리의 다른 컴포넌트들(예: week-navigator.tsx)과 코드 스타일이 불일치합니다. PR 목적이 접근성 속성 추가라면 포맷팅 변경은 diff 노이즈를 키우므로, 저장소에서 사용 중인 포매터/린트 규칙에 맞게 한쪽으로 통일하는 편이 좋습니다.

"use client";

import React, { useState, useId, type ReactNode } from "react";
import { ChevronDown, ChevronUp } from "lucide-react";
import { cn } from "@/lib/utils";

+      "sharp": ">=0.35.0",
+      "@hono/node-server": ">=2.0.5",


+      "sharp": ">=0.35.0",
+      "@hono/node-server": ">=2.0.5",


+      "hono": "^4.12.27",
+      "js-yaml": "^4.3.0"


+  sharp: '>=0.35.0'
+  '@hono/node-server': '>=2.0.5'


+  hono: ^4.12.27
+  js-yaml: ^4.3.0


+  sharp@0.35.3:
+    resolution: {integrity: sha512-ej0zVHuZGHCiABXcNxeYhpRnPNPAcvbG8RMdBAhDAxLKkCRVSpK3Iyu7qbqw3JMzoj0REeM6f3tJLtVwl0023Q==}
+    engines: {node: '>=20.9.0'}


seonghobae · 2026-08-05T12:52:48Z

Superseded by #266, which contains the same ContextSection decorative-chevron aria-hidden change inside the consolidated accessibility surface without this PR's unrelated package and lockfile drift. Closing the narrower stale lineage keeps one review path and avoids reintroducing dependency changes for a semantic-only accessibility fix.

google-labs-jules · 2026-08-05T12:54:42Z

Superseded by #266, which contains the same ContextSection decorative-chevron aria-hidden change inside the consolidated accessibility surface without this PR's unrelated package and lockfile drift. Closing the narrower stale lineage keeps one review path and avoids reintroducing dependency changes for a semantic-only accessibility fix.

Understood. Acknowledging that this work is superseded by #266 and stopping work on this task.

Copilot AI review requested due to automatic review settings July 22, 2026 20:31

Copilot started reviewing on behalf of seonghobae July 22, 2026 20:32 View session

Copilot AI reviewed Jul 22, 2026

View reviewed changes

Copilot AI review requested due to automatic review settings July 22, 2026 20:40

Copilot started reviewing on behalf of seonghobae July 22, 2026 20:41 View session

Copilot AI reviewed Jul 22, 2026

View reviewed changes

seonghobae closed this Aug 5, 2026

seonghobae mentioned this pull request Aug 5, 2026

🎨 Palette: [UX improvement] Add aria-hidden to decorative icons #266

Closed

Navigation Menu

Uh oh!

🎨 Palette: ContextSection 의 불필요한 Chevron 스크린리더 출력 제거 - #297

🎨 Palette: ContextSection 의 불필요한 Chevron 스크린리더 출력 제거#297
seonghobae wants to merge 2 commits into
developmentalfrom
palette-context-section-aria-14707259237627725677

seonghobae commented Jul 22, 2026

Uh oh!

google-labs-jules Bot commented Jul 22, 2026

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI left a comment

Uh oh!

seonghobae commented Aug 5, 2026

Uh oh!

google-labs-jules Bot commented Aug 5, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Uh oh!

Conversation

seonghobae commented Jul 22, 2026

Uh oh!

google-labs-jules Bot commented Jul 22, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

seonghobae commented Aug 5, 2026

Uh oh!

google-labs-jules Bot commented Aug 5, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants