Skip to content

feat(FileInput): improve component#6337

Merged
lisalupi merged 1 commit intomainfrom
fix/file-input-types
Apr 20, 2026
Merged

feat(FileInput): improve component#6337
lisalupi merged 1 commit intomainfrom
fix/file-input-types

Conversation

@lisalupi
Copy link
Copy Markdown
Collaborator

Summary

Type

  • Enhancement

Summarize concisely:

FileInput: various improvements to component:

  • new prop validator for custom errors
  • add default onChange for un formatted events
  • new prop disabledDragndrop to disabled drag and drop behavior
  • update prop onDrop to return acceptedFiles and errorFiles
  • new props onKeyDown, onKeyUp, onFocus, onBlur and name.
  • new prop onDelete for FileInput.List
  • When type = "dropzone" and size="small", the component is now a button and clicking anywere on it opens the file selector (previously, only clicking on the label opened the file selector)
  • Fix style issues

@lisalupi lisalupi requested a review from a team April 13, 2026 10:02
@lisalupi lisalupi self-assigned this Apr 13, 2026
@lisalupi lisalupi requested review from JulienSaguez and alexandre-combemorel and removed request for a team April 13, 2026 10:02
@lisalupi lisalupi added the enhancement New feature or request label Apr 13, 2026
@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Apr 13, 2026

🦋 Changeset detected

Latest commit: 817d722

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 3 packages
Name Type
@ultraviolet/ui Minor
@ultraviolet/form Patch
@ultraviolet/nextjs Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@codecov
Copy link
Copy Markdown

codecov bot commented Apr 13, 2026

Codecov Report

❌ Patch coverage is 88.33333% with 7 lines in your changes missing coverage. Please review.
✅ Project coverage is 92.60%. Comparing base (2b89546) to head (817d722).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
packages/ui/src/components/FileInput/index.tsx 87.71% 6 Missing and 1 partial ⚠️
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #6337      +/-   ##
==========================================
+ Coverage   83.30%   92.60%   +9.29%     
==========================================
  Files          44      533     +489     
  Lines         683    10036    +9353     
  Branches      195     3835    +3640     
==========================================
+ Hits          569     9294    +8725     
- Misses        104      677     +573     
- Partials       10       65      +55     
Files with missing lines Coverage Δ
...es/ui/src/components/FileInput/DropzoneContent.tsx 100.00% <100.00%> (ø)
...es/ui/src/components/FileInput/components/List.tsx 100.00% <100.00%> (ø)
packages/ui/src/components/FileInput/styles.css.ts 100.00% <100.00%> (ø)
packages/ui/src/components/FileInput/index.tsx 89.65% <87.71%> (ø)

... and 485 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2b89546...817d722. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@lisalupi lisalupi force-pushed the fix/file-input-types branch from 0286119 to 817d722 Compare April 20, 2026 13:28
@lisalupi lisalupi merged commit e4ae00d into main Apr 20, 2026
15 checks passed
@lisalupi lisalupi deleted the fix/file-input-types branch April 20, 2026 15:09
@scaleway-bot scaleway-bot mentioned this pull request Apr 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants