responsively-org / responsively-app

A modified web browser that helps in responsive web development. A web developer's must have dev-tool.
https://responsively.app
GNU Affero General Public License v3.0
22.77k stars 1.2k forks source link

Zoom in/out of devices on trackpad pinch gesture in the preview area #1049

Open manojVivek opened 1 year ago

prabhjotlamba1655 commented 1 year ago

Hi @manojVivek, I would like to work on this issue. Please assign it to me.

manojVivek commented 1 year ago

@prabhjotlamba1655 Please go ahead!

dgbkn commented 1 year ago

HI @manojVivek i would like to take up this issue if possible. Assign it to me.

manojVivek commented 1 year ago

@dgbkn Please go for it! We stopped doing user assignments for the issues.

Priyanshu72 commented 1 year ago

can you please elaborate this issue

MarCUS103 commented 1 year ago

@manojVivek Could you please elaborate the issue? It will be very beneficial while resolving the issue.

mukulpythondev commented 1 year ago

I am start working on this issue.

sameerbiradar2001 commented 1 year ago

Hi @manojVivek, I would like to work on this issue. Please assign it to me.

Alok-Singh-ops commented 1 year ago

@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.

manojVivek commented 1 year ago

@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.

Alok-Singh-ops commented 1 year ago

@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

manojVivek commented 1 year ago

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.

Alok-Singh-ops commented 1 year ago

@manojVivek Gotcha. By this entire preview section will be zoomed in/zoomed out

Alok-Singh-ops commented 1 year ago

@manojVivek Also, I have raised by PR kindly look into it

manojVivek commented 1 year ago

By this entire preview section will be zoomed in/zoomed out

Just clarifying, not the entire section, just the devices will be zoomed by the zoomFactor state value.

dompolochak commented 11 months ago

From what I can tell there needs to be some sort of onWheel or onScroll event to trigger the zoomFactor update but it doesnt seem to respond to any of those events (also doesn't respond to onClick) I think something about the webview is preventing the div from detecting events Edit: When I expand the div to be larger than the webview actions outside of the webview are detected so it is something about the webview not being able to detect the pinch to zoom

Screenshot 2023-12-04 at 2 59 37 PM
123upasana commented 10 months ago

Hi @manojVivek, I would like to work on this issue. Please assign it to me.

lyncasterc commented 9 months ago

Hey @manojVivek, I went ahead and created a PR for this (#1196) since it looks like it hasn't been addressed yet!

samr874 commented 8 months ago

@manojVivek explain me the functionality you are working on ?? which issue is this assign it to me

brkuhgk commented 7 months ago

Hello everyone, do we have any updates regarding this issue? If not, I'm more than willing to lend a hand and contribute. Thanks!

LungsomLamnio commented 3 months ago

No description provided.

can you please provide a description

Shashikant2707 commented 2 months ago

Hi @manojVivek, I would like to work on this issue

manojVivek commented 2 months ago

Please go ahead.