Skip to content

Add accepted-file-types-label attribute to dt-file-upload#196

Open
corsacca wants to merge 3 commits intomasterfrom
file-upload-MIME-categories
Open

Add accepted-file-types-label attribute to dt-file-upload#196
corsacca wants to merge 3 commits intomasterfrom
file-upload-MIME-categories

Conversation

@corsacca
Copy link
Copy Markdown
Member

Display user-friendly category labels (e.g., "Images, PDFs, Documents")
instead of raw MIME types in the upload zone hint text. The label is
passed from the theme via a new attribute, enabling translation support.

image

   Display user-friendly category labels (e.g., "Images, PDFs, Documents")
   instead of raw MIME types in the upload zone hint text. The label is
   passed from the theme via a new attribute, enabling translation support.
@netlify
Copy link
Copy Markdown

netlify bot commented Mar 23, 2026

Deploy Preview for jade-chebakia-17493f ready!

Name Link
🔨 Latest commit 86a5410
🔍 Latest deploy log https://app.netlify.com/projects/jade-chebakia-17493f/deploys/69ca33f14786c800088fb005
😎 Deploy Preview https://deploy-preview-196--jade-chebakia-17493f.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@corsacca corsacca requested a review from cairocoder01 March 23, 2026 10:42
@cairocoder01
Copy link
Copy Markdown
Collaborator

@corsacca This looks pretty good. Can you add a story for this in the stories file of this component? That helps for testing now and in the future as changes are made.

  - wire up acceptedFileTypesLabel in template args and HTML binding
  - add argType control for the new property
  - add AcceptedFileTypesLabel story showing friendly label override
@corsacca
Copy link
Copy Markdown
Member Author

@cairocoder01, good call. Story has been added

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.

2 participants