Skip to content

fix(classroom): お知らせ画面のヘッダー位置を他のクラス管理画面にそろえる (#1123)#1126

Open
smalruby3-editor-bot[bot] wants to merge 1 commit into
developfrom
topic/autopilot-1123
Open

fix(classroom): お知らせ画面のヘッダー位置を他のクラス管理画面にそろえる (#1123)#1126
smalruby3-editor-bot[bot] wants to merge 1 commit into
developfrom
topic/autopilot-1123

Conversation

@smalruby3-editor-bot

Copy link
Copy Markdown

Summary

クラス管理「お知らせ」全件一覧画面(teacher-notifications フェーズ)のヘッダー(パンくず + タイトル)位置を、他のクラス管理画面(評価画面 teacher-evaluation.jsx / 課題詳細 teacher-class-detail.jsx)とそろえた。

注: Issue の autopilot-base: topic/classroom-notification-center は指定時点のブランチだが、当該ブランチは既に develop へマージ・削除済み(お知らせ関連コードは develop 上に存在)。そのため本 PR の base は develop

原因

  • お知らせ一覧はパンくずをラッパー .detail-breadcrumbspadding: 1.25rem 2rem 0)で表示するが、その下の本文コンテナ .notifications-pagepadding: 1.25rem 2rem独自の top padding を持っていた。
  • 他の同型画面(.detail-layout)は「パンくずラッパーが top offset を持つので本文の top padding は 0」という規約(CSS コメントに明記)で、パンくず→タイトルの間隔が狭い。お知らせだけ余計に 1.25rem 空いていた。
  • さらにタイトル .notifications-page-title は他画面の .teacher-view-title と違い 太字(font-weight: bold)が無く、見出しが薄く見えていた。

Changes Made

  • .notifications-page: padding: 1.25rem 2rempadding: 0 2rem 1.25rem.detail-layout と同じ「上はパンくずラッパー任せ」の規約に統一)。左端オフセット(2rem)は不変。
  • .notifications-page-title: font-weight: bold を追加(.teacher-view-title と同じ濃い太字)。

CSS のみの変更で、DOM 構造・data-testid は一切変更していない。

Test Coverage

  • test/unit/components/teacher-notifications-list.test.jsx(6 件)が引き続き pass(data-testid 維持)。
  • npm run lint(scratch-gui は --max-warnings 0)が 0 エラー・0 警告 + Prettier OK。

補足(DoD スクリーンショット)

  • docs/classroom-notification-center/README.md のスクリーンショット表にはお知らせ全件一覧ページ単体のスクリーンショットが存在しない0102-notification-panel.png はドロップダウンパネルで、本 PR が変更するフルページのヘッダーとは別)。既存スクリーンショットは今回の変更の影響を受けない。
  • 一覧ページは先生ログイン(classroom バックエンド認証)が前提で headless 環境での撮影が難しいため、視覚確認は人間レビューでのお願いとしたい。CSS の値は既存規約(.detail-layout / .teacher-view-title)に一致させている。

Related Issues

Closes #1123

パンくずラッパー (.detail-breadcrumbs) が top offset を持つのに .notifications-page も

top padding を持っていたため、パンくずとタイトルの間隔が評価画面などより広かった。

.detail-layout と同じく top padding を 0 にし、タイトルを .teacher-view-title と同じ

濃い太字 (font-weight: bold) に揃えた。data-testid は不変で既存テストは通る。

Closes #1123

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@smalruby3-editor-bot smalruby3-editor-bot Bot added the 🤖 autopilot autopilot が処理対象にした Issue/PR(daemon が付与) label Jul 26, 2026
@smalruby3-editor-bot

smalruby3-editor-bot Bot commented Jul 26, 2026

Copy link
Copy Markdown
Author

🤖 autopilot status

field value
Status Review
AI Status
HITL Yes
Size small

Linked issue #1123. Maintained by autopilot (single writer); do not edit.

@github-actions

Copy link
Copy Markdown

🚀 Preview deployed: https://smalruby.jp/smalruby3-editor/topic/autopilot-1123/

@smalruby3-editor-bot

Copy link
Copy Markdown
Author

[Question] DoD の「docs/<feature>/screenshots/ の該当画面を撮り直す」が未対応です。

お知らせ全件一覧ページ(teacher-notifications フェーズ)のヘッダー位置・タイトル太字が視覚的に変わりますが、docs/classroom-notification-center/screenshots/ にこのフルページ単体のスクリーンショットが存在しないため撮り直し対象がありません(0102-notification-panel.png はドロップダウンパネルで別物)。この一覧ページは先生ログイン(classroom バックエンド認証)が前提で、autopilot の headless 環境では撮影できません。

対応方針: 人間レビューで実機の目視確認をお願いしたいです(パンくず→タイトル間隔が評価画面・課題詳細と揃っているか)。あわせて、このページのスクリーンショットを新規追加するか(documentation.md の DoD 上は追加が望ましい)はレビュアーの判断に委ねます。CSS 値自体は既存規約(.detail-layout / .teacher-view-title)と完全一致させているので、レイアウトの妥当性は担保されています。

@smalruby3-editor-bot

Copy link
Copy Markdown
Author

[FYI] タイトルを .teacher-view-title と揃える際、参照元にある overflow: hidden; text-overflow: ellipsis; white-space: nowrap;.notifications-page-title に付いていません。今回のヘッダー位置合わせ(padding / font-weight)の目的からは外れており、見出し文言(「お知らせ」/ "Notifications")が短く省略が起きないため実害はありません。将来的に完全なパリティを取るなら追加余地あり、というだけで対応不要です(コードは変更しません)。

@smalruby3-editor-bot

Copy link
Copy Markdown
Author

🤖 敵対的レビュー完了。指摘 2 件(Must 0 / Question 1 / FYI 1)。Must はなし。CSS のみの変更で、値は既存規約(.detail-layout / .teacher-view-title)と完全一致・.notifications-page / .notifications-page-title の使用箇所は一覧ページ 1 箇所のみ・data-testid 不変・unit test(6件) と prettier は pass を確認。Question/FYI はいずれも非ブロッキング。人間レビューへ。

@smalruby3-editor-bot smalruby3-editor-bot Bot added the 🙋 HITL 人間の対応待ち(Project HITL=Yes の投影) label Jul 26, 2026
@smalruby3-editor-bot
smalruby3-editor-bot Bot marked this pull request as ready for review July 26, 2026 14:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🤖 autopilot autopilot が処理対象にした Issue/PR(daemon が付与) 🙋 HITL 人間の対応待ち(Project HITL=Yes の投影)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix(classroom): お知らせ画面のヘッダー位置を他のクラス管理画面にそろえる

0 participants