Skip to content

Conversation

otukado
Copy link
Contributor

@otukado otukado commented Oct 1, 2025

なぜこの PR を入れたいのか

close: #3129

UI 変更部分のスクリーンショット

before

image

after

image

PR を出す前の確認事項

  • (機能の追加なら)追加することの合意がチームで取れている
    • 取れていない場合はチェックを外して PR にすれば OK
  • 動作確認ができている
  • 自分で一度コードを眺めて自分的に問題はなさそう

@otukado otukado requested a review from a team October 1, 2025 06:20
@otukado otukado self-assigned this Oct 1, 2025
@otukado otukado added the enhancement New feature or request label Oct 1, 2025
Copy link

github-actions bot commented Oct 1, 2025

@otukado otukado moved this from Inbox to In review in traQ Client Oct 1, 2025
Copy link

codecov bot commented Oct 1, 2025

Codecov Report

❌ Patch coverage is 0% with 13 lines in your changes missing coverage. Please review.
✅ Project coverage is 13.55%. Comparing base (83506c9) to head (081faee).

Files with missing lines Patch % Lines
...gationBar/NavigationContent/ClipFoldersElement.vue 0.00% 13 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #4865      +/-   ##
==========================================
- Coverage   13.55%   13.55%   -0.01%     
==========================================
  Files         704      704              
  Lines       31646    31658      +12     
  Branches      676      676              
==========================================
  Hits         4290     4290              
- Misses      27344    27356      +12     
  Partials       12       12              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

name="bookmark"
mdi
:class="$style.icon"
:data-is-selected="$boolAttr(isSelected)"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

data-is-selected の代わりに aria-selected を使って,aria-role='tab'aria-controls を指定するといいかも

Copy link
Contributor

@uni-kakurenbo uni-kakurenbo Oct 7, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

aria-controls まで指定しようとすると少々面倒 (PR のスコープ外の差分が大きくなってしまう) かもだから無視でもいいもしれない

@otukado otukado requested a review from a team October 9, 2025 04:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

Status: In review

2 participants