Skip to content

スタンプの文字の位置を修正#9619

Merged
machida merged 2 commits intomainfrom
hotfix/stamp
Feb 5, 2026
Merged

スタンプの文字の位置を修正#9619
machida merged 2 commits intomainfrom
hotfix/stamp

Conversation

@machida
Copy link
Copy Markdown
Member

@machida machida commented Feb 5, 2026

Summary by CodeRabbit

  • スタイル

    • レポートのスタンプタイトル領域がコンテナ幅全体に対応するよう改善されました。
  • テスト

    • システムテストでリンク存在チェックをより明確なアサーションに置き換え、「活動時間を登録してください。」のリンク挙動を検証。
    • フォーム送信後に表示される成功メッセージ「ユーザー情報を更新しました。」を追加で確認するようになりました。

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Feb 5, 2026

📝 Walkthrough

Walkthrough

スタイルの変更とシステムテストのアサーション置換を行いました。.stamp__content.is-titlewidth: 100%; を追加し、システムテストでは has_link?/has_no_link?assert_link/assert_no_link に置き換え、フォーム送信後の成功メッセージを追加で検証しています。

Changes

Cohort / File(s) Summary
CSS スタイル更新
app/assets/stylesheets/application/blocks/report/_stamp.css
.stamp__content.is-titlewidth: 100%; を追加(高さは既存の height: 32%; のまま)。
システムテスト更新
test/system/home/required_fields_test.rb
has_link?/has_no_link?assert_link/assert_no_link に置換。フォーム送信後に "ユーザー情報を更新しました。" の成功メッセージを追加でアサート。

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

Suggested reviewers

  • komagata

Poem

🐰 横に伸びて、タイトルに晴れ間。
幅を満たして、すっきりと。
テストは確かめ、結果ににっこり。
小さな変更で、気分はふわり。

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Description check ⚠️ Warning プルリクエストの説明が未記入です。リポジトリのテンプレートで要求されているセクション(Issue、概要、変更確認方法、Screenshot)が全く含まれていません。 プルリクエストの説明を追加し、Issue番号、変更内容の概要、確認方法、変更前後のスクリーンショットを含める必要があります。
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed プルリクエストのタイトルは「スタンプの文字の位置を修正」であり、CSS変更ファイルの内容と一致する主な変更を説明しています。
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
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch hotfix/stamp

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 removed the request for review from okuramasafumi February 5, 2026 09:58
- フォーム送信後にリダイレクト完了を待機するアサーションを追加
- assert has_link?/has_no_link? を assert_link/assert_no_link に変更
@machida machida merged commit 519b2d2 into main Feb 5, 2026
3 checks passed
@machida machida deleted the hotfix/stamp branch February 5, 2026 11:10
@github-actions github-actions bot mentioned this pull request Feb 5, 2026
2 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