Skip to content

reset系CSSを削除し、Tailwindのresetを使うようにした#9555

Merged
machida merged 1 commit intomainfrom
reset
Jan 28, 2026
Merged

reset系CSSを削除し、Tailwindのresetを使うようにした#9555
machida merged 1 commit intomainfrom
reset

Conversation

@machida
Copy link
Copy Markdown
Member

@machida machida commented Jan 28, 2026

Summary by CodeRabbit

リリースノート

  • スタイル

    • ページコンテンツヘッダーのレスポンシブ動作と間隔を調整しました。
    • サイドナビゲーションの表示ルールを複数の画面サイズに対応させました。
    • ユーザーアイコン表示のマージンを改善しました。
  • チア

    • CSSリセットルールセットの依存関係を削除しました。
    • Tailwindユーティリティの不要な読み込みを廃止しました。
    • スタイルシート構成を最適化しました。

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

@machida machida removed the request for review from okuramasafumi January 28, 2026 03:55
@machida machida self-assigned this Jan 28, 2026
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Jan 28, 2026

📝 Walkthrough

Walkthrough

CSSの初期化・リセットファイル(_reset.css、_ress.css)と、Tailwindユーティリティのインポートを削除し、複数のコンポーネントのレスポンシブデザインをメディアクエリで再構成しました。

Changes

コホート / ファイル 変更概要
CSS初期化・リセット削除
app/assets/stylesheets/initializers/_reset.css, app/assets/stylesheets/initializers/_ress.css, app/assets/stylesheets/initializers/_tailwind.css, app/assets/stylesheets/common-imports.css
グローバルCSSリセット規則(_reset.css、_ress.css)を完全削除。Tailwindユーティリティのインポートを削除。common-imports.cssから2つのインポート行を削除。
レスポンシブデザイン再構成
app/assets/stylesheets/application/blocks/page-content/_page-content-header-actions.css, app/assets/stylesheets/atoms/_a-side-nav.css, app/assets/stylesheets/shared/blocks/_footprints.css
基本スタイルからスペーシングとフレックス宣言を削除し、メディアクエリ内に移動。新しいブレークポイント(max-width: 39.9375em、max-width: 47.9375em)を導入してレスポンシブ動作を調整。

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

Suggested reviewers

  • okuramasafumi
  • komagata

Poem

🐰 リセット削除し、新たなる調べ
レスポンシブなメディアで整える
Tailwindの幽霊も退場して
シンプルなカスケードが踊る
スタイルの春がやってきた ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Description check ⚠️ Warning PRの説明が完全に記入されていません。テンプレートで必須とされているIssue番号、概要、変更確認方法、スクリーンショットのセクションがすべて不足しています。 テンプレートに従い、Issue番号、変更内容の概要、確認方法、該当する場合はスクリーンショットを追加してください。
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed PRタイトルは変更内容を正確に反映しており、reset系CSSの削除とTailwindのresetへの移行という主要な変更を明確に説明しています。
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.


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.

@machida machida merged commit 1ae2719 into main Jan 28, 2026
5 checks passed
@machida machida deleted the reset branch January 28, 2026 04:26
@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