-
Notifications
You must be signed in to change notification settings - Fork 140
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix(DropZone): ドラッグ&ドロップ時に選択したファイルがFormDataに格納されるようにした #4964
Conversation
commit: |
(めちゃくちゃ些細なPRタイトルの話なのですが、Conventional commitsではスコープを型の後ろに |
@@ -86,6 +86,9 @@ export const DropZone = forwardRef<HTMLInputElement, DropZoneProps & ElementProp | |||
overrideEventDefault(e) | |||
setFilesDraggedOver(false) | |||
onSelectFiles(e, e.dataTransfer.files) | |||
if (fileRef.current) { | |||
fileRef.current.files = e.dataTransfer.files | |||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
良さそうに見える。これまで問題になってなかったてことは、現状のユースケースだと onSelectFiles
で State で受け取ってる感じなんですかねぇ。
form から直接やりたいってのは server action 用途とかなんでしょうか?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
form から直接やりたいってのは server action 用途
ですね!
現状のユースケースだと onSelectFiles で State で受け取ってる感じ
たぶんそんな感じですかねー
そもそもファイルをアップロードするケースがあんまり少ないのかもしれませんが
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍️ 良さそう。これで既存機能がぶっ壊れるということは流石にないだろうしあったら使い方のほうが悪いはず…。
大丈夫と信じてマージします! |
関連URL
概要
DropZone
コンポーネントで、ドラッグ&ドロップで選択されたファイルがinput
タグのDOMに格納されず、form
で送信した際に意図したファイルがFormData
オブジェクトに格納されない問題がありました。変更内容
DropZone
コンポーネントで、ドラッグ&ドロップで選択されたファイルがinput
タグのDOMにデータが格納されるようにした。確認方法
pnpm add https://pkg.pr.new/kufu/smarthr-ui@4964
で変更したパッケージをインストールし、DropZone
でファイルをドラッグ&ドロップしてからform
をsubmit
して正常に選択したファイルが送信されていること。