動画一覧ページにタグ一覧が重なってしまうのを修正した

[machida]

Summary by CodeRabbit

リリースノート

• スタイル
  • ムービーページにサイドナビゲーションを追加しました。
  • ムービーリストの表示レイアウトを改善しました。
  • ページネーションを一覧の下部に移動しました。

[coderabbitai]

Warning

Rate limit exceeded

@machida has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 18 minutes and 44 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: da37fcde-6b17-44b3-9f7a-404f47961d17

📥 Commits

Reviewing files that changed from the base of the PR and between 2f7d08b and 591e734.

📒 Files selected for processing (1)

• test/test_helper.rb

📝 Walkthrough

Walkthrough

2つのムービービューのテンプレートを更新し、.page-body__inner要素にhas-side-navクラスを追加。2番目のテンプレートではコンテンツ構造をリファクタリングし、ムービーアイテムのネストされたコンテナと改善されたページネーション配置を導入。

Changes

コホート / ファイル(s)	変更概要
ムービービューのスタイリング更新 app/views/movies/index.html.slim, app/views/practices/movies/index.html.slim	.page-body__inner要素にhas-side-navモディファイアクラスを追加し、サイドナビゲーションレイアウトに対応。
ムービーリストのコンテンツ構造化 app/views/practices/movies/index.html.slim	.page-content.is-movies配下に.movies__itemsと.movie-listの新しいネストされた構造を導入し、ムービーパーシャルのレンダリングを移行。ページネーション要素をコンテンツの下部に再配置。

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

• 動画一覧にタグ一覧と絞り込み機能を追加 #8590: app/views/movies/index.html.slimを修正する別のPRで、タグベースのサイドナビゲーション追加と関連するビュー変更が含まれており、今回のコミットと密接に関連。

Suggested reviewers

• komagata

Poem

🐰 サイドナビが育ち、レイアウト新しく
ムービーたちは整列、ネスト深く
ページネーション踏み、下へと移ろい
テンプレート整う、スタイルも麗し
ウサギも満足、デザイン調和す ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Description check	⚠️ Warning	プルリクエストの説明が完全に空の状態で、テンプレートの必須セクション（Issue番号、概要、変更確認方法、スクリーンショット）がすべて未記入です。	Issue番号、変更の概要、確認方法、変更前後のスクリーンショットをテンプレートに従って記入してください。

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title check	✅ Passed	プルリクエストのタイトルは、動画一覧ページにタグ一覧が重なる問題の修正という変更内容と一致しており、具体的で明確です。
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 unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch fix-movies

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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