feat(gui): お知らせパネルの ⋯ メニューと一覧ページのパンくず/戻る (#1111 レビュー)#1119
Merged
smalruby3-editor-bot[bot] merged 2 commits intoJul 26, 2026
Merged
Conversation
人間レビュー指摘への対応:
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>
|
🚀 Preview deployed: https://smalruby.jp/smalruby3-editor/feature/notification-menu-1111/ |
敵対レビュー FYI 対応: - use-teacher-notifications.js の docstring が旧「開くと全既読」のまま だったので実挙動(⋯「すべて既読にする」で明示既読)に更新 - .notifications-panel の overflow/max-height を外し、ヘッダーの ⋯ ポップアップが短いパネルでクリップされないように(プレビューは最大 5 件で高さ有界なのでパネルスクロールは不要) Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Author
敵対レビュー結果 — MERGEABLE(Must なし)diff 全体を検証(jest 28件 / admin 43件パス、eslint/prettier クリーン、locale ペア 8/8)。主要リスク(handleMarkAllRead の stale-closure、⋯ メニューの二重発火、自動既読廃止による二重カウント、messages.title のスコープ、seeAll 削除の残参照)はいずれも問題なしと確認。 FYI 2 件は対応(コミット e10be71):
|
smalruby3-editor-bot
Bot
merged commit Jul 26, 2026
2d99863
into
topic/classroom-notification-center
1 check passed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
お知らせ通知センター(#1111)への追加レビュー指摘に対応。
1) パネルに ⋯(三点)メニュー — 一覧を件数に関係なく開ける
handleMarkAllReadを hook に追加。in-memory readAt / localStorage キャッシュ / バッジを更新)。2) 全件一覧ページのナビゲーション(ポリシー化)
Test
classroom/ui-ux.md§1.8 更新、集約スクショ 0102 を ⋯ メニュー付きで撮り直しRefs #1111(レビュー指摘)
🤖 Generated with Claude Code