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

whiteboard image cannot be downloaded #925

Closed
andreicon opened this issue Nov 23, 2023 · 2 comments
Closed

whiteboard image cannot be downloaded #925

andreicon opened this issue Nov 23, 2023 · 2 comments

Comments

@andreicon
Copy link

andreicon commented Nov 23, 2023

Description


After drawing in the whiteboard, if the user tries to download a copy of the canvas to the local filesystem by clicking on the download button, the file download won't be triggered. The same issue may occur when the user tries to download files that were shared via chat (not sure if this is a feature).

Current behavior


File download is not triggered.

Expected Behavior


The file should be downloaded (i.e. save dialog opens, allowing the user to save a copy of the file to disk)

Possible Solution


Allowing downloads via the sandbox attribute of the iframe.

Steps to reproduce


  • Join a call
  • Open the whiteboard
  • Draw something
  • click "download image"

Environment details


The sandbox feature for the iframe was introduced in this PR: #861
Unless there's a reason for the omission, the list of permissions should also include allow-downloads

Opened a PR with a fix: #926

@saghul
Copy link
Member

saghul commented Nov 23, 2023

Have you tested that is enough? In Electron one needs to provide the UI and stuff for downloads.

@andreicon
Copy link
Author

Fixed by #926. closing.

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

No branches or pull requests

2 participants