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

Add screenshot option to @uppy/screen-capture #3310

Open
hpvd opened this issue Nov 9, 2021 · 7 comments
Open

Add screenshot option to @uppy/screen-capture #3310

hpvd opened this issue Nov 9, 2021 · 7 comments
Labels

Comments

@hpvd
Copy link

hpvd commented Nov 9, 2021

Screencast function is great!

...but
a) sometimes it is "too much" for the user
b) does not give the possibility to provide different types of "screencast" to the user

solution:

split "screencast" to 4 types

  1. simple screenshot,
  2. screencast,
  3. screencast with sound
  4. pure sound

Details:
a) sometimes it is "too much"
for the user and it adds

  • unnecessary upload size when only a screenshot is needed (image instead of video)
  • unnecessary/ partial misleading browser alerts "allow this website to use your microphone"

b) yes, a) could be partly solved with
https://uppy.io/docs/screen-capture/#displayMediaConstraints ...
...but it prevents things like providing different types of "screencast" to the user, to trigger /motivate him to upload the best possible for the receiver, e.g. in a complex bug report:
a video of the process, some screenshots of details, additional spoken hints

edit: just noticed there are plans for audio only:
#1840 (comment)

edit 2: maybe it's not only about splitting screencast, but providing the 4 types of upload independently...

@Murderlon
Copy link
Member

Hi, I think we already offer enough granularity.

unnecessary upload size when only a screenshot is needed (image instead of video)

I think most users would opt for using their OS to take a screenshot of their screen. Although I'm open to adding this option to screen capture as an extra button.

unnecessary/ partial misleading browser alerts "allow this website to use your microphone"

I forgot whether you're microphone sound is added in the screen capture but I can't test currently because of this bug I found: #3311. If it's not, then we shouldn't ask for that permission indeed.

@giannik
Copy link

giannik commented Nov 14, 2021

having an option in screencast for screenshotonly would be great.
something like this :
https://codesandbox.io/s/webrtc-screenshot-it3d0?from-embed=&file=/src/index.js

then image would be added to dashboard for user to manage with the image editor plugin.

@Murderlon Murderlon changed the title split screencast to 4 different types Add screenshot option to @uppy/screen-capture Nov 15, 2021
@giannik
Copy link

giannik commented Jan 19, 2022

Hello ,
Is there some way to sponsor this feature for adding a screnshot option, so that it gets in ?

@colinambler
Copy link

I'd be really interested in the image-only screenshot feature too. Anything we can do get it rolling?

@Murderlon
Copy link
Member

We don't have a system in place to sponsor individual features, unfortunately. We do have commercial support for Uppy. Being a Transloadit customer with commercial support could turn this into a priority, but currently we are focusing on Uppy 3.0, a new Uppy website, and some other things. So from our perspective this is not priority at the moment.

@arturi arturi removed the Triage label Jul 6, 2023
@herringtown
Copy link

Hi @Murderlon , any additional inertia on this from the product side (cc: @colinambler, @giannik, @hpvd )? I would love to have this as well, as the screencast is a bit overkill and I really just need a way to capture a screenshot. Absent that, if someone has figured out how to pass in the correct MediaTrackConstraints to achieve this, would love to know how they did it...

@Murderlon
Copy link
Member

Unfortunately with less maintainers and more pressing issues this is not a priority right now. If you need a quick, I would gladly review / help out with a PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

6 participants