feat(gui,admin): 通知の日次取得・アバターメニュー・白ベル・全件ページ (#1111 レビュー)#1118
Merged
smalruby3-editor-bot[bot] merged 3 commits intoJul 26, 2026
Merged
Conversation
人間レビュー指摘への対応:
1) コスト削減: お知らせ取得を 60 秒ポーリング → 1 日 1 回
- use-teacher-notifications: interval 廃止。localStorage に日付つきで
キャッシュし、その日はじめてのクラス管理オープン時だけ 1 回取得。
同日再オープンは API を叩かない。共有 PC 対策で先生メールで識別。
2) 右上をアバターメニューに(teacher / admin 両方)
- メール頭文字(k / kt)の丸 + ▼。クリックでポップアップ(メール +
ログアウト。将来の設定項目もここに集約)。紫背景で視認できる白丸 +
紫文字。initialsFromEmail を lib に切り出し。旧 email/logout は廃止
- フルスクリーン Modal は左「戻る」で閉じ右上に × が無いため、アバターを
右端(1.25rem)へ寄せて余白をなくした(admin と同じ・追加レビュー指摘)
3) 通知ベルを白一色 SVG に(絵文字は主張が強い)。アバターの左へ
4) パネルは先頭 5 件 + 「すべて見る(N件)」→ 全件一覧ページ
(teacher-notifications フェーズ・10 件/ページ・追加 API なし)
docs: classroom/ui-ux.md §1.8 更新、集約ページのスクショ 0101/0102 を
新 UI で撮り直し。
検証: jest 119件 + headless で teacher UX 11/11・admin アバター・右端
寄せ(gap 20px)を確認。gui/admin lint クリーン。
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
アバターメニュー化・白ベル・右端寄せ後の UI に合わせて、 docs/classroom-notification-center/screenshots を一括更新 (teacher タイトルバー / admin ヘッダーとも新 chrome)。 Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
|
🚀 Preview deployed: https://smalruby.jp/smalruby3-editor/feature/notification-ux-cost/ |
- [Must] mesh 回帰スクリプト修復: ログアウトがアバターメニュー内へ 移動したため、mesh-v2-classroom-binding.mjs で classroom-avatar-button を先にクリックしてから classroom-teacher-logout を押すよう修正 - [Question] dev-bypass 等 email 不明(空キー)時はキャッシュを使わず 毎回取得(共有 PC で別の先生の通知が混ざらないように)。本番の Google/Microsoft ID トークンは email を持つのでキャッシュは有効 - [Question] docs/classroom/testing.md の testid を更新(logout は アバターメニュー内・classroom-teacher-email 廃止・avatar-* 追加) - [FYI] admin app.css の未使用 .admin-header-email を削除 Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Author
敵対レビュー結果(自動レビュー + 対応済み)diff 全体を正確性・セキュリティ・rules 準拠・整合性・テストで検証(scratch-gui 28件 / admin 43件パス、lint/prettier クリーン)。 指摘と対応(コミット 0d0d9af)
クリーン確認(抜粋)
判定: 指摘対応後 MERGEABLE |
smalruby3-editor-bot
Bot
merged commit Jul 26, 2026
dd61119
into
topic/classroom-notification-center
9 checks 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)への人間レビュー指摘に対応。コスト削減と右上 UI の刷新が主眼。
Changes
1) コスト削減:通知取得を 60 秒ポーリング → 1 日 1 回
use-teacher-notifications: 60 秒 interval を廃止。localStorage に日付つきキャッシュし、その日はじめてクラス管理を開いたときだけGET /notificationsを 1 回呼ぶ。同日再オープンは API を叩かない。共有 PC 対策で先生メールでキャッシュを識別。2) 右上をアバターメニューに(teacher / admin 両方)
kouji@…→K/kouji.takao@…→KT)の丸 + ▼。クリックでポップアップ(メール + ログアウト。将来の設定項目もここに集約)。紫背景で視認できる白丸 + 紫文字。3) 通知ベルを白一色 SVG に(絵文字 🔔 は主張が強い)。アバターの左へ。
4) パネルは先頭 5 件 + 「すべて見る(N件)」→ 全件一覧ページ
teacher-notificationsフェーズ・10 件/ページのクライアントページネーション(追加 API なし=コスト増なし)。docs
classroom/ui-ux.md§1.8 更新、集約ページのスクショ 0101/0102 を新 UI で撮り直し。AWS コスト観点(ご依頼のレビュー)
Test
avatar-initials/teacher-avatar-menu/teacher-notifications-list/ 日次取得フック)Refs #1111(レビュー指摘)
🤖 Generated with Claude Code