Skip to content
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

Q-File's "max-file-size" and "max-total-size" props don't work. #15668

Closed
userpta005 opened this issue Apr 3, 2023 · 3 comments
Closed

Q-File's "max-file-size" and "max-total-size" props don't work. #15668

userpta005 opened this issue Apr 3, 2023 · 3 comments

Comments

@userpta005
Copy link

What happened?

Q-File's "max-file-size" and "max-total-size" props don't work, in all cases it doesn't pass validation, even putting a smaller file than declared in "max-file-size " and "max-total-size" .

It is possible to test in the documentation itself.
https://quasar.dev/vue-components/file-picker#restricting-files

What did you expect to happen?

Expected to pass validation when inserting files smaller than the size declared in "max-file-size " and "max-total-size".

Reproduction URL

https://quasar.dev/vue-components/file-picker#restricting-files

How to reproduce?

Just try to insert a file smaller than the one declared in "max-file-size " and "max-total-size".

Flavour

Quasar CLI with Vite (@quasar/cli | @quasar/app-vite)

Areas

Components (quasar)

Platforms/Browsers

Firefox

Quasar info output

No response

Relevant log output

No response

Additional context

No response

@github-actions
Copy link

github-actions bot commented Apr 3, 2023

Hi @Ozzy005! 👋

It looks like you provided an invalid or unsupported reproduction URL.
Do not use any service other than Codepen, jsFiddle, StackBlitz, Codesandbox, and GitHub.
Make sure the URL you provided is correct and reachable. You can test it by visiting it in a private tab, another device, etc.
Please edit your original post above and provide a valid reproduction URL as explained.

Without a proper reproduction, your issue will have to get closed.

Thank you for your collaboration. 👏

@pdanpdan
Copy link
Collaborator

pdanpdan commented Apr 3, 2023

Works for me

@pdanpdan pdanpdan closed this as completed Apr 3, 2023
@userpta005
Copy link
Author

it really works, I hadn't seen that the values are in bytes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants