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: copy to clipboard on non secure context (no SSL) #521

Merged
merged 1 commit into from Oct 17, 2022

Conversation

Anteus
Copy link
Contributor

@Anteus Anteus commented Oct 16, 2022

Solves #492

Adds a fallback for the Clipboard API because it's not supported on non SSL environments.
Confirmed to be working on Firefox.

PR Checklist

  • I've started from master
  • I've only committed changes related to this PR
  • All Unit tests pass
  • I've removed all commented code
  • I've removed all unneeded console.log statements

@WDaan
Copy link
Collaborator

WDaan commented Oct 17, 2022

Is this confirmed to be working?

@Anteus
Copy link
Contributor Author

Anteus commented Oct 17, 2022

Yes it works on Firefox and iOS.

@WDaan WDaan changed the title copy non secure context fix: copy to clipboard on non secure context (no SSL) Oct 17, 2022
@WDaan WDaan merged commit 058766f into VueTorrent:master Oct 17, 2022
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.

None yet

2 participants