Skip to content

開発者オーバレイの位置精度表示を整数に揃える#5581

Merged
TinyKitten merged 1 commit intodevfrom
codex/fix-dev-overlay-accuracy-floor
Mar 11, 2026
Merged

開発者オーバレイの位置精度表示を整数に揃える#5581
TinyKitten merged 1 commit intodevfrom
codex/fix-dev-overlay-accuracy-floor

Conversation

@TinyKitten
Copy link
Member

@TinyKitten TinyKitten commented Mar 11, 2026

概要

  • 開発者オーバレイの位置精度表示で小数点以下を切り捨てて整数表示に変更
  • 位置精度の小数点切り捨てを確認するテストを追加

確認

  • npm run lint
  • npm run typecheck
  • npm test -- DevOverlay.test.tsx --runInBand

Summary by CodeRabbit

バグ修正

  • ロケーション精度の表示値が小数点以下で切り捨てられるようになりました。例えば、精度15.9mは「15m」と表示されるよう改善されました。

@github-actions github-actions bot added the react label Mar 11, 2026
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Mar 11, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: ca60d643-2299-4421-9ec4-902d35ef0179

📥 Commits

Reviewing files that changed from the base of the PR and between 7103dab and 5e9bbb1.

📒 Files selected for processing (2)
  • src/components/DevOverlay.test.tsx
  • src/components/DevOverlay.tsx

📝 Walkthrough

説明書

位置情報の精度値を表示する前に床関数(floor)を使用して小数以下を切り下げ、15.9が「15m」として表示されるようにするロジックを追加。DevOverlay コンポーネントとそのテストで実装され、精度値の正規化に対応。

変更内容

コホート / ファイル 概要
位置情報精度の床関数処理
src/components/DevOverlay.tsx
null安全で床処理され負の値に対応した accuracyMeters 変数を導入し、2つの LOCATION ACCURACY メトリックカードで直接的な精度値の使用を置き換え。
位置情報精度テスト
src/components/DevOverlay.test.tsx
精度値 15.9 が「15m」として表示されることを検証するテストを追加し、床関数の動作を確認。

推定コード査読時間

🎯 2 (Simple) | ⏱️ ~10 minutes

関連する可能性のあるPR

提案されるラベル

react

🐰 精度値よ、床で整えよ
小数は消えて、整数のみ
15.9も15へと、すっきり表示
ウサギのテストが確かめてくれる
メートルの表示、今日も正確に!

🚥 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 The title clearly and concisely describes the main change: converting the developer overlay's location accuracy display from decimal to integer values by flooring the value.
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 codex/fix-dev-overlay-accuracy-floor

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

@TinyKitten TinyKitten self-assigned this Mar 11, 2026
@TinyKitten TinyKitten merged commit f7deda7 into dev Mar 11, 2026
7 checks passed
@TinyKitten TinyKitten deleted the codex/fix-dev-overlay-accuracy-floor branch March 11, 2026 02:33
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