Skip to content

feat(gui): お知らせパネルの ⋯ メニューと一覧ページのパンくず/戻る (#1111 レビュー)#1119

Merged
smalruby3-editor-bot[bot] merged 2 commits into
topic/classroom-notification-centerfrom
feature/notification-menu-1111
Jul 26, 2026
Merged

feat(gui): お知らせパネルの ⋯ メニューと一覧ページのパンくず/戻る (#1111 レビュー)#1119
smalruby3-editor-bot[bot] merged 2 commits into
topic/classroom-notification-centerfrom
feature/notification-menu-1111

Conversation

@smalruby3-editor-bot

Copy link
Copy Markdown

Summary

お知らせ通知センター(#1111)への追加レビュー指摘に対応。

1) パネルに ⋯(三点)メニュー — 一覧を件数に関係なく開ける

  • 参考 UI(Facebook)準拠。パネルヘッダー右に ⋯ メニュー: すべて既読にする(未読 0 で無効)/ お知らせを開く(全件一覧ページへ)。
  • 旧「5 件以上のときだけ下部に すべて見る」を廃止 → 件数に関わらず ⋯「お知らせを開く」で一覧に到達(不便さの解消)。
  • 開くだけでは既読にしない(バッジ維持)。既読は ⋯「すべて既読にする」で明示(handleMarkAllRead を hook に追加。in-memory readAt / localStorage キャッシュ / バッジを更新)。

2) 全件一覧ページのナビゲーション(ポリシー化)

  • パンくずを 「クラス管理 > お知らせ」 に(先頭「クラス管理」でいつでもトップ=クラス一覧へ戻れる)。
  • キャンセルが不適切な画面なので 左下に「戻る」ボタン(トップへ)。

Test

  • jest: teacher-notifications / -list / hook / avatar 28 件 + admin 43 件
  • headless 8/8(開くだけでは未読維持・⋯ から既読/一覧・<5 件でも一覧到達・パンくず「クラス管理」・左下戻る)
  • gui / admin lint クリーン
  • docs: classroom/ui-ux.md §1.8 更新、集約スクショ 0102 を ⋯ メニュー付きで撮り直し

Refs #1111(レビュー指摘)

🤖 Generated with Claude Code

人間レビュー指摘への対応:

1) パネルヘッダーに ⋯(三点)メニューを追加(参考 UI 準拠)
   - 「すべて既読にする」(未読 0 で無効)/「お知らせを開く」(全件一覧)
   - 一覧は件数に関係なく開ける(旧「5 件以上のときだけ すべて見る」を廃止)
   - パネルを開くだけでは既読にせず、既読は ⋯「すべて既読にする」で明示
     (handleMarkAllRead を hook に追加。in-memory readAt/キャッシュ/バッジを更新)

2) 全件一覧ページのナビゲーション(ポリシー化)
   - パンくずを「クラス管理 > お知らせ」に(先頭「クラス管理」でトップ=
     クラス一覧へ戻れる)。testid classroom-breadcrumb-top
   - キャンセルが不適切な画面なので左下に「戻る」ボタン
     (classroom-notifications-back → トップへ)

docs: ui-ux §1.8 の testid/挙動を更新、集約スクショ 0102 を ⋯ メニュー
つきで撮り直し。

検証: jest(notifications/list/hook/avatar 28件 + admin 43件)+ headless
8/8(開くだけでは未読維持・⋯ から既読/一覧・<5件でも一覧到達・パンくず
「クラス管理」・左下戻る)。gui/admin lint クリーン。

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@github-actions

Copy link
Copy Markdown

敵対レビュー FYI 対応:
- use-teacher-notifications.js の docstring が旧「開くと全既読」のまま
  だったので実挙動(⋯「すべて既読にする」で明示既読)に更新
- .notifications-panel の overflow/max-height を外し、ヘッダーの ⋯
  ポップアップが短いパネルでクリップされないように(プレビューは最大
  5 件で高さ有界なのでパネルスクロールは不要)

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@smalruby3-editor-bot

Copy link
Copy Markdown
Author

敵対レビュー結果 — MERGEABLE(Must なし)

diff 全体を検証(jest 28件 / admin 43件パス、eslint/prettier クリーン、locale ペア 8/8)。主要リスク(handleMarkAllRead の stale-closure、⋯ メニューの二重発火、自動既読廃止による二重カウント、messages.title のスコープ、seeAll 削除の残参照)はいずれも問題なしと確認。

FYI 2 件は対応(コミット e10be71):

  • フック docstring が旧「開くと全既読」のままだったのを実挙動に更新
  • パネルの overflow/max-height を外し、ヘッダー ⋯ ポップアップが短いパネルでクリップされないように(プレビュー最大 5 件で高さ有界)

@smalruby3-editor-bot
smalruby3-editor-bot Bot merged commit 2d99863 into topic/classroom-notification-center Jul 26, 2026
1 check passed
@smalruby3-editor-bot
smalruby3-editor-bot Bot deleted the feature/notification-menu-1111 branch July 26, 2026 12:18
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.

0 participants