Skip to content

意味もなくヘッダーのレイアウトが機種依存していた#5384

Merged
TinyKitten merged 1 commit intodevfrom
fix/numbering-position
Feb 20, 2026
Merged

意味もなくヘッダーのレイアウトが機種依存していた#5384
TinyKitten merged 1 commit intodevfrom
fix/numbering-position

Conversation

@TinyKitten
Copy link
Member

@TinyKitten TinyKitten commented Feb 20, 2026

Summary by CodeRabbit

リリースノート

  • リファクタリング
    • 複数のコンポーネントからプラットフォーム固有の条件分岐を削除しました。
    • コンポーネントの動作をすべてのプラットフォームで統一しました。
    • コンポーネント処理ロジックを簡潔にしました。

@TinyKitten TinyKitten self-assigned this Feb 20, 2026
@github-actions github-actions bot added the react label Feb 20, 2026
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 20, 2026

No actionable comments were generated in the recent review. 🎉


📝 Walkthrough

Walkthrough

複数のヘッダーコンポーネントと NumberingIconSquare から Platform の条件分岐を削除し、transformOrigin プロパティを固定値「bottom」または「center」に統一するリファクタリング。プラットフォーム固有の処理を廃止し、コンポーネント間での統一的な動作を実現。

Changes

Cohort / File(s) Summary
ヘッダーコンポーネントの Platform 除去
src/components/HeaderE235.tsx, src/components/HeaderJL.tsx, src/components/HeaderJRWest.tsx, src/components/HeaderTY.tsx
react-native から Platform インポートを削除。NumberingIcon の transformOrigin を Platform.OS === 'android' ? 'bottom' : undefined から常に 'bottom' に変更。
HeaderSaikyo の簡素化
src/components/HeaderSaikyo.tsx
NumberingIcon の transformOrigin を Platform 条件分岐から固定値 'bottom' に統一。
NumberingIconSquare のデフォルト値統一
src/components/NumberingIconSquare.tsx
transformOrigin プロパティのデフォルト値をプラットフォーム依存 (Android 時は center、それ以外は bottom) から統一的に 'center' に変更。

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related PRs

Suggested labels

react

Poem

🐰 Platform のチェック外し、

transformOrigin 統一へ、

すべてのウサギに bottom の調和,

プラットフォーム超えて、同じ道を~♪

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed PRタイトルは、複数のヘッダーコンポーネントにおけるプラットフォーム依存の条件分岐を削除し、NumberingIconのtransformOriginを統一する変更の主要目的を正確に表現しています。
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 (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/numbering-position

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

@TinyKitten TinyKitten merged commit 0dc77c5 into dev Feb 20, 2026
7 checks passed
@TinyKitten TinyKitten deleted the fix/numbering-position branch February 20, 2026 20:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant