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

webapp+raw_server: tell browsers to always download a file if it should be downloaded #983

Merged
merged 1 commit into from
Oct 4, 2016

Conversation

haraldschilly
Copy link
Contributor

fixes #982

testing:

  • try it with a pdf or svg file
  • webapp recompilation
  • restart the project to pick up the modified smc-project/raw_server.coffee file

@haraldschilly
Copy link
Contributor Author

possible addition: avoid forcing downloads for html pages, by not adding ?download to the url for their URLs. (came up in #984)

@williamstein
Copy link
Contributor

Tested. This does work for png files now (etc.), though it didn't work (with this sort of exception) when my ad blocker (ublock) extension was enabled. But we obviously can't workaround extensions. So seems good to me.

@williamstein williamstein merged commit 75c149d into master Oct 4, 2016
@haraldschilly haraldschilly deleted the issue/982 branch November 27, 2018 21:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

fix the download button to always download a file
2 participants