feat(FileUpload): add `previewFile` prop by mdotme · Pull Request #5935 · nuxt/ui
📝 Walkthrough
Walkthrough
A new optional previewFile prop is added to the FileUpload component to control whether file previews are generated and displayed. The createObjectUrl function's return type is updated from string to string | undefined to reflect that it may not generate a blob URL. When previewFile is false, the function returns undefined; otherwise, it creates and returns a blob URL as before. Minor default value adjustments accompany the new prop.
Estimated code review effort
🎯 2 (Simple) | ⏱️ ~8 minutes
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
| Check name | Status | Explanation |
|---|---|---|
| Description check | ✅ Passed | The description is related to the changeset, indicating the PR resolves issue #5918 and documents the type of change (enhancement and new feature). |
| Linked Issues check | ✅ Passed | The PR fully addresses issue #5918 by implementing a previewFile prop to disable image previews, allowing consistent placeholder behavior across file types. |
| Out of Scope Changes check | ✅ Passed | All changes are directly related to implementing the previewFile prop feature requested in issue #5918; no out-of-scope modifications detected. |
| Docstring Coverage | ✅ Passed | No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. |
| Title check | ✅ Passed | The title accurately and concisely summarizes the main change: adding a new previewFile prop to the FileUpload component. |
✏️ Tip: You can configure your own custom pre-merge checks in the settings.
✨ Finishing touches
- 📝 Generate docstrings
🧪 Generate unit tests (beta)
- Create PR with unit tests
- Post copyable unit tests in a comment
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.
Comment @coderabbitai help to get the list of available commands and usage tips.