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

Is setting a specific zoom level part of the MVP? #44

Open
youennf opened this issue Oct 25, 2024 · 2 comments
Open

Is setting a specific zoom level part of the MVP? #44

youennf opened this issue Oct 25, 2024 · 2 comments

Comments

@youennf
Copy link

youennf commented Oct 25, 2024

The current proposal allows to set a particular zoom level, which requires getting the current zoom level.
This seems ok to implement for tab capture, but might prove more difficult for window surfaces.

AIUI, the main usecase is to zoom in or zoom out, which is something many OS/applications support.

This makes me wonder whether, instead of exposing the ability to set a specific zoom level, we could reduce to allow to zoom-in/zoom-out for the MVP. It seems these primitives could be easier to implement for both tabs and windows.

Would we lose much in terms of functionality?

@eladalon1983
Copy link
Member

Would we lose much in terms of functionality?

IIUC, the proposal here is to only support increase/decrease and get rid of everything else - getter, setter, supported-levels, event handelr. That means:

  • Apps won't be able to display the current zoom-level to the user over the preview tile. (And to listen to out-of-band changes to this value.)
  • Users would lose the ability to "reset" the zoom level with a single-step, sending the level back to 100%.
  • Apps won't be able to grey out controls at min/max value.
  • Apps won't be able to please the user with more advance controls such as drop-down lists or sliders.
  • Multi-step transitions of zoom level would require multiple clicks from the user...
  • ...and multiple layout updates by the captured application, instead of a single zoom-change and subsequent layout change.

@eladalon1983 eladalon1983 transferred this issue from w3c/mediacapture-screen-share-extensions Nov 13, 2024
@eladalon1983 eladalon1983 changed the title [Capture control] Is setting a specific zoom level part of the MVP? Is setting a specific zoom level part of the MVP? Nov 13, 2024
@eladalon1983
Copy link
Member

Issue transferred; heads up to those discussion participants who might otherwise be looking for it elsewhere: @jan-ivar, @youennf

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