Skip to content

tailwindのユーティリティクラスを優先させた#9566

Merged
machida merged 5 commits intomainfrom
tailwind-important
Jan 28, 2026
Merged

tailwindのユーティリティクラスを優先させた#9566
machida merged 5 commits intomainfrom
tailwind-important

Conversation

@machida
Copy link
Copy Markdown
Member

@machida machida commented Jan 28, 2026

Summary by CodeRabbit

  • スタイル
    • 複数のビューでユーティリティクラス表記を統一・明示化し、重要度指定(!プレフィックス)を導入してレイアウトと見た目の一貫性を改善しました(表示への影響のみ)。
  • テスト
    • システムテストの手順と検証を安定化するためにページ遷移やセレクタ検証を調整しました。

✏️ Tip: You can customize this high-level summary in your review settings.

@github-actions github-actions bot requested a review from komagata January 28, 2026 09:27
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Jan 28, 2026

📝 Walkthrough

Walkthrough

複数のSlimテンプレートとシステムテストで、Tailwind系ユーティリティクラス表記をドット記法から明示的な class='...' 属性へ移行し、多数のユーティリティに !(important)プレフィックスを追加。機能やデータフローの変更はなし。

変更内容

コホート / ファイル(s) 変更内容
管理画面:招待URL
app/views/admin/invitation_url/index.html.slim
Slimのドット記法を class='...' に置換。複数ユーティリティを ! 付きにしてレイアウトの重要度を上書き。ボタン高さ等に ! を追加。
ユーザ一覧/テーブル
app/views/admin/users/_table.html.slim
テーブルセルのユーティリティを ! 付きに変更(text-*whitespace-* 等)。サブスクリプション列のリンクに rel="noopener"data-subscription-id を追加し、リンク内にスピナーを挿入。
共通ヘッダー / トーク表示
app/views/application/header/_header.html.slim, app/views/talks/show.html.slim
コンテナ/行のクラス表記を class='...' へ統一し、必要箇所に ! を追加。
ユーザ学習時間関連テンプレート
app/views/users/_learning_time_frames.html.slim, app/views/users/form/_learning_time_frames.html.slim
テーブル・行・セルのユーティリティを class='...' に移行し、重要度付きユーティリティ(!)に置換。セルの name 属性や内部 span 構成を整理。
ウェルカム/ロゴ関連
app/views/welcome/choose_courses.html.slim, app/views/welcome/logo.html.slim
多数のグリッド/スペーシングラッパーを明示的 div + class='...' に展開し、グリッド・間隔・マージン等のユーティリティを ! 付きで上書き。大きめのマークアップ展開あり。
システムテスト
test/system/current_user/learning_time_frame_test.rb, test/system/require_login/users_test.rb
DOM検証を page.has_selector? から assert_selector に変更。users_test.rb はヘルパー置換で明示的な Capybara ステップへ(visit + assert_selector + assert_no_text)。

Sequence Diagram(s)

(条件に合わないため省略)

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20分

Possibly related PRs

Suggested reviewers

  • komagata
  • okuramasafumi
  • su-su-su-su

ポエム

🐰✨ クラスにビックリマークを添えて
ピョンと整えたマークアップ道
グリッドもセルもスッと並び直し
ウサギは跳ねてCSSの野を駆ける🥕
よくやったね、今日もおつかれさま。

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Description check ⚠️ Warning プルリクエストの説明が提供されていません。リポジトリのテンプレートで要求されている Issue 番号、概要、変更確認方法、スクリーンショットなどの情報が完全に欠けています。 PR の説明を追加し、Issue 番号、変更内容の概要、変更確認方法、必要に応じてスクリーンショットを含めてください。
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed PR タイトルはプルリクエストの主な変更内容(Tailwind ユーティリティクラスに !important フラグを追加して優先度を上げる)を正確に表現しており、簡潔で明確です。
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

name変数を使わずに直接インラインで属性を設定するように変更。
- users_test: 具体的なセレクタを使用してページロード完了を確実に待機
- learning_time_frame_test: テーブルのレンダリング完了を待機、不要なwithinとscroll_toを削除
@machida machida force-pushed the tailwind-important branch from 0970549 to d27d6e6 Compare January 28, 2026 12:51
@machida machida merged commit f8a3b86 into main Jan 28, 2026
3 checks passed
@machida machida deleted the tailwind-important branch January 28, 2026 13:06
@github-actions github-actions bot mentioned this pull request Jan 28, 2026
27 tasks
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.

1 participant