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

Zoom is being reset when trying to move around the canvas #26

Open
Mheaus opened this issue Apr 30, 2024 · 11 comments
Open

Zoom is being reset when trying to move around the canvas #26

Mheaus opened this issue Apr 30, 2024 · 11 comments

Comments

@Mheaus
Copy link

Mheaus commented Apr 30, 2024

Hello 👋

I've been using the new version of inspect with the last release of xstate and react for a few months now.
I've noticed slow performance and issues when trying to move the view accross the canvas.

When I'm moving the view with a trackpad or by zooming, it'll lag and reset the zoom to fit the window.

It seems to refresh too much?

I hope this little video will help you identify the issue and if I can help in any way please let me know 🙏 .

Enregistrement.de.l.ecran.2024-04-30.a.12.53.25.mov
@davidkpiano
Copy link
Member

Thanks for the report and the video - it definitely helps. Is there a way to share the machine (either a link or the exported code)?

@Mheaus
Copy link
Author

Mheaus commented Apr 30, 2024

Sadly it's on a private project, but I can work on a repro !
I'll let you know.

@cpetzel
Copy link

cpetzel commented May 1, 2024

This happens for us as well. It seems to reset the zoom whenever the inspector receives an event. In our case, it receives dozens of events per second, so the zoom resetting makes the inspector almost unusable.

@davidkpiano
Copy link
Member

This happens for us as well. It seems to reset the zoom whenever the inspector receives an event. In our case, it receives dozens of events per second, so the zoom resetting makes the inspector almost unusable.

Understood. We will have a fix for this soon.

@davidkpiano
Copy link
Member

Sorry for the delay here; still working on this. We are overhauling the canvas layer which will introduce many performance improvements and features.

@Mheaus
Copy link
Author

Mheaus commented May 31, 2024

Sorry for the delay here; still working on this. We are overhauling the canvas layer which will introduce many performance improvements and features.

No problem, take your time. 👍
Can I help in any way?

@WIStudent
Copy link

Any updates on this issue? My machine is quite big, I cannot read anything without zooming in. The frequent resetting of the zoom makes the inspector almost impossible to use.

I am starting to regret upgrading to v5. Is there a way to use the old viszalizer with v5 until the issues with the new inspector are resolved?

@okowalow
Copy link

okowalow commented Dec 5, 2024

I am experiencing the same issue: my machine isn't that big (photo below)
Zrzut ekranu 2024-12-5 o 13 59 29

but when I try to zoom and move around the canvas, it resets to default. + it's very slow. It's much slower than xstate visualizer for xstate 4. Previous inspector was really fast

@davidkpiano
Copy link
Member

I want to rebuild the inspector in React Flow, and open-source it. If anyone wants to help contribute in this effort, please let me know.

@Mheaus
Copy link
Author

Mheaus commented Dec 5, 2024

I'd be very much interested in helping

@aldotms70
Copy link

I want to rebuild the inspector in React Flow, and open-source it. If anyone wants to help contribute in this effort, please let me know.
The inspector is a very useful tool but it is unusable. I would be happy to contribute. Let me known the link to the new project. Thanks

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

6 participants