Skip to content

feat: add Filepicker Dropzone component with interactive examples #6

Merged
phev8 merged 2 commits intomainfrom
file-picker
Feb 25, 2026
Merged

feat: add Filepicker Dropzone component with interactive examples #6
phev8 merged 2 commits intomainfrom
file-picker

Conversation

@phev8
Copy link
Contributor

@phev8 phev8 commented Feb 25, 2026

No description provided.

@greptile-apps
Copy link

greptile-apps bot commented Feb 25, 2026

Greptile Summary

Adds a new filepicker dropzone component with comprehensive documentation and refactors interactive examples into modular files.

Key Changes:

  • New FilepickerDropzone component with drag-and-drop file upload using react-dropzone
  • Introduced Field, FieldLabel, FieldDescription, and FieldError components for form field composition
  • Added Separator component using Radix UI
  • Refactored all interactive examples from single file into separate modular files with centralized exports
  • Comprehensive documentation with React Hook Form integration example

Issues Found:

  • Uses experimental React.useEffectEvent API which is not available in React 19.2.4 (requires useCallback instead)

Confidence Score: 2/5

  • This PR requires fixing the experimental React API usage before merging
  • The component uses React.useEffectEvent which is experimental and not available in the current React version (19.2.4), causing a runtime error. Once replaced with useCallback, the implementation is solid with good accessibility, error handling, and documentation.
  • Pay close attention to registry/radix-nova/filepicker-dropzone.tsx - the experimental API must be replaced

Important Files Changed

Filename Overview
registry/radix-nova/filepicker-dropzone.tsx New filepicker dropzone component with experimental React API usage (useEffectEvent)
components/docs/interactive-examples/filepicker-dropzone.tsx Clean interactive example demonstrating the filepicker dropzone component
components/ui/field.tsx New field wrapper components for form elements with label, description, and error handling
components/ui/separator.tsx New separator component using Radix UI primitives

Last reviewed commit: 9e899d4

Copy link

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

15 files reviewed, 1 comment

Edit Code Review Agent Settings | Greptile

…allback, and useMemo in FilepickerDropzone component
@phev8 phev8 merged commit fd660af into main Feb 25, 2026
2 checks passed
@phev8 phev8 deleted the file-picker branch February 25, 2026 08:19
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