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

Fix paste issue with fileuploader #3702

Merged
merged 1 commit into from
Jun 19, 2015
Merged

Fix paste issue with fileuploader #3702

merged 1 commit into from
Jun 19, 2015

Conversation

timcooper
Copy link
Contributor

Client reported this one, they were trying to copy from Excel into input fields (title etc.) but on paste fileuploader was kicking in thinking they were trying to upload something and popping dialogs up.

There is a similar fix in place already over here: https://github.com/bolt/bolt/blob/release/2.2/app/src/js/upload-files.js#L188

@timcooper
Copy link
Contributor Author

I think this is the right approach for a 2.2 hotfix, however I would suggest for 2.3/master that the underlying fileuploader lib is updated instead. In the latest version this option defaults to null (technically undefined, but behaviour is same - blueimp/jQuery-File-Upload@d60f53c)

@GwendolenLynch
Copy link
Contributor

I would suggest for 2.3/master that the underlying fileuploader lib is updated instead

👍 I've wanted this for ages myself as I have a couple of things I have to work around myself.

@rarila
Copy link
Contributor

rarila commented Jun 17, 2015

I thought I already did here in my overhauling of src/app/ I have in the works here. But it seems it's still the old one. I probably add that to that PR as it seems we can move it from lib to bower now.

PS: it was jquery-ui I updated

rossriley added a commit that referenced this pull request Jun 19, 2015
@rossriley rossriley merged commit f845427 into bolt:release/2.2 Jun 19, 2015
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.

4 participants