Skip to content

UploadControl: Added setAllowedType - #172

Closed
JanGalek wants to merge 1 commit into
nette:masterfrom
JanGalek:master
Closed

UploadControl: Added setAllowedType#172
JanGalek wants to merge 1 commit into
nette:masterfrom
JanGalek:master

Conversation

@JanGalek

@JanGalek JanGalek commented Dec 6, 2017

Copy link
Copy Markdown
  • bug fix? no
  • new feature? yes
  • BC break? no
  • doc PR: nette/docs#??? - if it will merged I will send PR

Added new method to UploadControl, whitch set mimeTypes and attribute accept.

@JanTvrdik

Copy link
Copy Markdown
Contributor

@JanGalek

JanGalek commented Dec 6, 2017

Copy link
Copy Markdown
Author

@JanTvrdik ou, ok I will remove js, and rebase.

Firefox 👎

And add attribute accept with mime types

Removed empty line

Fix: condition to check type

Fix: conding-standard
@JanGalek

JanGalek commented Dec 6, 2017

Copy link
Copy Markdown
Author

@JanTvrdik https://bugzilla.mozilla.org/show_bug.cgi?id=373621, fixed ?

I tried js log and firefox (57.0) return same type as chrome

@JanTvrdik

Copy link
Copy Markdown
Contributor

@JanGalek The commit is from 2014 and the issue is resolved. But it's still sth that need's to be considered.

@JanGalek

JanGalek commented Dec 6, 2017

Copy link
Copy Markdown
Author

@JanTvrdik ok, and if we Dtect firefox <= 50 and type needed is application/pdf than we allow application/x-download with check extension ?

Problem was with PDF right ?

@dg

dg commented Feb 6, 2018

Copy link
Copy Markdown
Member

What about to add attribute accept, when rule MIME_TYPE is used? The same way as i.e. attribute range is added in TextInput::addRule().

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.

3 participants