Skip to content

Conversation

@Qs-F
Copy link
Contributor

@Qs-F Qs-F commented Oct 7, 2025

関連URL

概要

  • InputFileの中身をネイティブのinputとmultiplyAppendableの挙動をするものに分割した。
  • multiplyAppendableが再度ファイルを選択できない問題を解消した。
  • 自動テストで担保できない部分を担保するため、マニュアルテスト用のStorybookを作成した。

変更内容

  • InputFileの中身をネイティブのinputとmultiplyAppendableの挙動をするものに分割した。
    • 共通化できるコンポーネントもあるが、そもそもrefなどが入り組んでおりメンテナンスするたびに壊れるような状態のため分割してメンテナビリティの向上を期待している。
    • 共通化やhooks化は気持ちのある人に任せます。
  • multiplyAppendableが再度ファイルを選択できない問題を解消した。
  • 自動テストで担保できない部分を担保するため、マニュアルテスト用のStorybookを作成した。
    • 自動テストにはDataTransferが実装されていないため、ファイルを扱うようなテストがほぼ書けない

確認方法

@misako0927 さんとモブプロで修正しています。

以下の4つをInformationPanelの説明に従って操作してみる

@pkg-pr-new
Copy link

pkg-pr-new bot commented Oct 7, 2025

Open in StackBlitz

npm i https://pkg.pr.new/kufu/smarthr-ui@5864

commit: 2741e97

@Qs-F Qs-F changed the title chore(InputFile): InputFileをNativeと独自に分割 fix(InputFile): InputFileをinputの挙動に寄せ、フォーム内でデータが適切に処理できるように修正 Oct 7, 2025
@Qs-F Qs-F marked this pull request as ready for review October 7, 2025 12:51
@Qs-F Qs-F requested a review from a team as a code owner October 7, 2025 12:51
@Qs-F Qs-F requested review from moshisora and schktjm and removed request for a team October 7, 2025 12:51
Copy link
Contributor

@masa0527 masa0527 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

手動のstoryも確認して動作的に良さそうかと思いました!

@Qs-F Qs-F merged commit 74219b1 into master Oct 22, 2025
16 checks passed
@Qs-F Qs-F deleted the fix/inputfile-divide-into-native branch October 22, 2025 12:20
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.

4 participants