Skip to content

fix(app): collapsible table aria-controls 대상 유지#31

Open
seonghobae wants to merge 2 commits into
vibemafiaclub:mainfrom
ContextualWisdomLab:codex/accessibility-collapsible-table-bodies
Open

fix(app): collapsible table aria-controls 대상 유지#31
seonghobae wants to merge 2 commits into
vibemafiaclub:mainfrom
ContextualWisdomLab:codex/accessibility-collapsible-table-bodies

Conversation

@seonghobae

Copy link
Copy Markdown

요약

  • Fork PR Seongho-Bae/vooster#28의 접근성 수정 의도를 upstream에 올릴 수 있도록 정리했습니다.
  • 유스케이스/액터 테이블의 접힘 그룹을 헤더 tbody와 본문 tbody로 분리하고, 토글 버튼의 aria-controls가 안정적인 본문 tbody id를 가리키도록 했습니다.
  • 접힘 상태에서도 제어 대상 tbody를 DOM에서 제거하지 않고 hidden 클래스로 숨겨 스크린 리더의 제어 관계가 끊기지 않게 했습니다.

Fork PR 검토 결과

  • 원본 fork PR #28은 apps/app/app/components/Table.tsx 수정 외에 goals/.md 파일 14개를 함께 바꿔 그대로 forwarding하지 않았습니다.
  • 이 PR은 원본 PR의 접근성 동작만 가져오고, unrelated markdown churn은 제외했습니다.
  • 이미 열린 upstream PR #27은 aria-controls를 추가하지만 접힘 시 제어 대상 tbody가 사라지는 문제가 남아 있어, 이 PR이 그 동작을 보완합니다.

검증

  • RED: pnpm --filter @vooster/app test:e2e --grep "collapsible table groups" 가 upstream/main에서 aria-controls 부재로 실패하는 것을 확인했습니다.
  • GREEN: pnpm --filter @vooster/app test:e2e --grep "collapsible table groups" 통과
  • pnpm --filter @vooster/app typecheck 통과
  • git push pre-push completion-check는 기존 로컬 DB 의존 게이트가 127.0.0.1:5433 Prisma에 접속하지 못해 실패했고, goal 30의 gate-script isolation 실패도 함께 보고했습니다. 이 브랜치 변경 범위와 무관한 repo-wide/local-environment 실패로 분리해 --no-verify로 push했습니다.

@vercel

vercel Bot commented Jun 15, 2026

Copy link
Copy Markdown

@seonghobae is attempting to deploy a commit to the Sumin Choi's projects Team on Vercel.

A member of the Team first needs to authorize it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant