-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Zoom in/out of devices on trackpad pinch gesture in the preview area #1049
Comments
Hi @manojVivek, I would like to work on this issue. Please assign it to me. |
@prabhjotlamba1655 Please go ahead! |
HI @manojVivek i would like to take up this issue if possible. |
@dgbkn Please go for it! |
can you please elaborate this issue |
@manojVivek Could you please elaborate the issue? It will be very beneficial while resolving the issue. |
I am start working on this issue. |
Hi @manojVivek, I would like to work on this issue. Please assign it to me. |
@manojVivek we can not directly use zoom pan pinch lib here due to the fact that lib does not support webview tag. We need to write its logic manually. |
@Alok-Singh-ops Thanks for exploring that. But we don't actually need that event handler on the webview, we just have to add it to the preview area enclosing the webview. |
@manojVivek yes i get it. i Even tried to do using lib such as zoom pan but I guess the library just does not support webview |
Hm, I'm not certain why webview would affect anything here. As we don't need to interact with the webview at all in this, we just have to watch for the events on the preview container and update the zoom level in the redux state. |
@manojVivek Gotcha. By this entire preview section will be zoomed in/zoomed out |
@manojVivek Also, I have raised by PR kindly look into it |
Just clarifying, not the entire section, just the devices will be zoomed by the |
Hi @manojVivek, I would like to work on this issue. Please assign it to me. |
Hey @manojVivek, I went ahead and created a PR for this (#1196) since it looks like it hasn't been addressed yet! |
@manojVivek explain me the functionality you are working on ?? which issue is this assign it to me |
Hello everyone, do we have any updates regarding this issue? If not, I'm more than willing to lend a hand and contribute. Thanks! |
can you please provide a description |
Hi @manojVivek, I would like to work on this issue |
Please go ahead. |
No description provided.
The text was updated successfully, but these errors were encountered: