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

Consideration of proposal to support zoom in/out #40

Open
eladalon1983 opened this issue Nov 8, 2024 · 1 comment
Open

Consideration of proposal to support zoom in/out #40

eladalon1983 opened this issue Nov 8, 2024 · 1 comment

Comments

@eladalon1983
Copy link
Member

eladalon1983 commented Nov 8, 2024

If I understood @youennf correctly, he proposes to replace setZoomLevel(value) with zoomIn() and zoomOut(). (Or a similar shape.)

  1. Did I understand correctly?
  2. What are the benefits of this approach? Could they be articulated by someone who supports that change?
  3. Assuming that this approach has some benefits over setZoomLevel(value) in some use cases, could a compromise be achieved, serving both use cases, using either of the following?
    • Specifying both setZoomLevel(value) and zoomIn()/zoomOut()
    • Specifying setZoomLevel() as accepting long or ZoomAction, where ZoomAction is an enum consisting of "in" and "out"
@eladalon1983 eladalon1983 transferred this issue from w3c/mediacapture-screen-share-extensions Nov 13, 2024
@eladalon1983 eladalon1983 changed the title [Capture control] Consideration of proposal to support zoom in/out Consideration of proposal to support zoom in/out Nov 13, 2024
@eladalon1983
Copy link
Member Author

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

1 participant