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

Adds Video and Audio preview #455

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

victorwads
Copy link

@victorwads victorwads commented Jun 5, 2022

Preview Video:
image

Preview Audio:
Captura de Tela 2022-06-05 às 14 36 21

Copy link
Contributor

@Bellisario Bellisario left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems good, just a little missing...

client/scripts/ui.js Outdated Show resolved Hide resolved
@victorwads
Copy link
Author

Thanks @Bellisario, fixed

@fm-sys
Copy link
Contributor

fm-sys commented Jun 9, 2022

@victorwads can you add some screenshots, so that we can take a look at the changes? The past has shown that PR's containing screenshots are more likely getting merged... 😄

@victorwads
Copy link
Author

Done @fm-sys

@dylanjamesdev
Copy link

I love this!

@seanmavley
Copy link

Looking forward to this being merged.

@Jeyso215
Copy link

Jeyso215 commented Jul 29, 2023

@victorwads merge it here: https://github.com/WhateverItWorks/snapdrop-reworked

Owner seems to not maintaining pull request anymore

@zhuanghaotao
Copy link

zhuanghaotao commented Dec 2, 2023 via email

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.

8 participants