Skip to content

fix: improve recipe metadata filtering and unit handling#116

Merged
amemya merged 8 commits into
mainfrom
feat/issue-113-recipe-filter
Jun 25, 2026
Merged

fix: improve recipe metadata filtering and unit handling#116
amemya merged 8 commits into
mainfrom
feat/issue-113-recipe-filter

Conversation

@amemya

@amemya amemya commented Jun 25, 2026

Copy link
Copy Markdown
Owner

フィルムモードでプルダウンが正常に動かない問題を修正
ソート順や絞り込みアルゴリズムの改善も含む

fixed #113

@coderabbitai

coderabbitai Bot commented Jun 25, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: b014c4e7-aab4-4b62-9369-045912642d12

📥 Commits

Reviewing files that changed from the base of the PR and between e159f1a and 35592a2.

📒 Files selected for processing (2)
  • frontend/src/components/MetadataSettingsPanel.tsx
  • frontend/src/components/ToggleInput.tsx

📝 Walkthrough

Summary by CodeRabbit

  • 機能改善
    • 候補リストの表示順が、より自然な並びになるよう改善されました。
    • フィルム関連の候補が、入力内容に応じて段階的に絞り込まれるようになりました。
  • バグ修正
    • 数値として扱えない入力の表示が、入力時の不要な空白を除いた形で統一されました。
    • 候補選択時の入力欄の挙動を調整し、提案値を選びやすくしました。

Walkthrough

MetadataSettingsPanel でフォーマットのフォールバックがトリム済み入力に統一され、レシピ候補の抽出・並び替え条件が更新されました。ToggleInput は一時表示状態を持ち、候補表示時の入力値、変更、失焦点処理を切り替えるようになりました。

Changes

メタデータ入力の表示調整

Layer / File(s) Summary
整形フォールバックの統一
frontend/src/components/MetadataSettingsPanel.tsx
formatFocalLengthformatApertureformatShutterSpeedformatISOformatTempformatTime の未変換時の返却値が trimmed に変わった。
レシピ候補の抽出と並び替え
frontend/src/components/MetadataSettingsPanel.tsx
parseValuecustomSort が追加され、希釈・時間・stock の値比較に基づく候補ソートと、選択値の存在に応じた段階的フィルタが入った。
提案入力の一時表示
frontend/src/components/ToggleInput.tsx
tempValue 状態と hasSuggestions 判定が追加され、入力の onMouseDownonChangeonBlurvalue/list/datalist の扱いが更新された。

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed 主要な変更であるレシピメタデータの絞り込み・単位処理の改善を適切に表しています。
Description check ✅ Passed プルダウン不具合の修正とソート・絞り込み改善に言及しており、変更内容と一致しています。
Linked Issues check ✅ Passed #113の「一部プルダウンが出ない不具合」と挙動の統一に対し、候補表示と絞り込み改善で対応できています。
Out of Scope Changes 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 docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/issue-113-recipe-filter

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.

@amemya amemya merged commit 03c020b into main Jun 25, 2026
4 checks passed
@amemya amemya deleted the feat/issue-113-recipe-filter branch June 25, 2026 11:16
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