Skip to content

fix(admin): 詳細画面の戻るボタンのスタイルを揃える (#1106 レビュー)#1116

Merged
smalruby3-editor-bot[bot] merged 2 commits into
topic/classroom-notification-centerfrom
fix/admin-back-button-style
Jul 26, 2026
Merged

fix(admin): 詳細画面の戻るボタンのスタイルを揃える (#1106 レビュー)#1116
smalruby3-editor-bot[bot] merged 2 commits into
topic/classroom-notification-centerfrom
fix/admin-back-button-style

Conversation

@smalruby3-editor-bot

Copy link
Copy Markdown

Summary

人間レビュー指摘(Admin > みんなの課題 > 詳細)への対応。「← 一覧に戻る」ボタンが class 無しの素の <button> で、既定ブラウザ表示のまま他のボタン(タブ・アクション)から浮いていた。

Changes

  • 共通クラス .admin-back-button(白背景・1px 枠・角丸・padding = アクションボタンと同じ)を app.css に追加
  • 同じ素ボタンが全ビューに 6 箇所あったので一括で揃えた: みんなの課題詳細 / バグ報告詳細 / クラス詳細 / 期限切れ復元(プラン・alive・完了)

Test

  • admin SPA: lint + jest 42 件パス(既存 testid は不変)
  • dev bypass で実画面確認:白背景・角丸でタブ/「再公開する」等と統一(computed: 1px solid / radius 6.4px / bg white

Refs #1106(レビュー指摘)

🤖 Generated with Claude Code

smalruby3-editor-bot Bot and others added 2 commits July 26, 2026 00:02
みんなの課題/バグ報告/クラス・課題/期限切れ復元の各詳細画面の
「← 一覧に戻る」が class 無しの素の button で、既定ブラウザ表示のまま
浮いていた。共通の .admin-back-button (白背景・1px枠・角丸=アクション
ボタンと同じ見た目) を追加し、6 箇所すべてに適用。

人間レビュー指摘 (Admin > みんなの課題詳細) への対応。同じ素ボタンが
他ビューにも全部あったので一括で揃えた。

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
人間レビュー指摘 2 点:
- 前コミットの .admin-back-button に font: inherit を付けたため UA 既定
  (13.3px) より大きくなり他ボタンより一回り大きく見えていた。
  .admin-actions button とセレクタを統合し font 未指定=UA 既定に揃えた
  (実測: 両者 font 13.33px / padding 6.4×14.4 / 高さ 32.8px で一致)
- みんなの課題詳細で限定公開項目の authorName/schoolLevel/subject 等が
  'undefined' と表示されていた。限定公開 (#1109) はこれらを省略できる
  ため欠損ではなく未入力。orUnspecified で undefined→「未指定」に
  フォールバック (詳細のクレジット行・属性行・一覧の投稿者)

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

Copy link
Copy Markdown

@smalruby3-editor-bot
smalruby3-editor-bot Bot merged commit 5344302 into topic/classroom-notification-center Jul 26, 2026
9 checks passed
@smalruby3-editor-bot
smalruby3-editor-bot Bot deleted the fix/admin-back-button-style branch July 26, 2026 09:55
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