Fix/search_input_height

[Pugma]

User description

変更の根拠

• height 指定の取りやめ
  • アセンダー・ディセンダーが本来の文字の高さからはみ出しているため、文字の高さで input を指定するとはみ出てしまう
  • とりあえず下手に高さを抑えないことで対処
• line-height
  • 行の高さを調整するためのものなので、1 行しかないここでは不要だと判断

参考画像です

ディセンダーに関しては分かりやすいと思う
ちょうど文字が切れる位置で線が引かれているはずだし

---

PR Type

Bug fix, Enhancement

---

Description

• .inputWrapper の line-height プロパティを削除しました。単一行の入力には不要なためです。
• .input の height プロパティを 1em から 100% に変更し、テキストのクリッピング問題を解消しました。
• defineProps を Vue 3.5 の仕様に合わせてデフォルト値を持つ分割代入に変更しました。
• iconSize 計算で props.size の代わりに分割代入された size 変数を使用するように調整しました。

---

Changes walkthrough 📝

Relevant files

Enhancement

SearchInput.vue Adjusted input height and props handling in SearchInput component src/components/UI/SearchInput.vue Removed line-height property from .inputWrapper as it is unnecessary for single-line inputs. Changed height property of .input from 1em to 100% to avoid text clipping issues. Updated defineProps to use destructuring with default values, aligning with Vue 3.5 conventions. Adjusted iconSize computation to use destructured size variable instead of props.size. +8/-15

---

💡 PR-Agent usage: Comment /help "your question" on any pull request to receive relevant information

[github-actions]

Preview (prod) → https://257-prod.portfolio-preview.trapti.tech/
Preview (dev) → https://257-dev.portfolio-preview.trapti.tech/

[github-actions]

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 2 🔵🔵⚪⚪⚪

🧪 No relevant tests

🔒 No security concerns identified

⚡ Recommended focus areas for review デフォルト値の設定 definePropsを使用しているが、Vue 3.5の仕様に合わせたデフォルト値の設定が適切か確認が必要です。 スタイル変更 .inputのheightプロパティを100%に変更したことの影響を確認する必要があります。

[github-actions]

PR Code Suggestions ✨

No code suggestions found for the PR.

[ogu-kazemiya]

確認しました〜
文字の上下が切れないようになってます
propsの分割代入も問題ないです
ありがとうございます！