-
Notifications
You must be signed in to change notification settings - Fork 211
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
Make the UI mobile-friendly #17
Comments
Do you feel the play mode isn't mobile friendly ? Or only the edit mode ? |
Hey @1000i100, thanks for the reply! It's been a while so I've just had another play around with it to try and provide some more concrete examples for you... Here's a couple of the major symptoms:
This all stems from, I believe, two issues:
If the sidebar made better use of the screen by, for example, sliding in across the entire screen and being dismissable, then I believe the bulk of these issues would be solved. There is a separate case to be made about allowing the whole canvas to be resizable as I believe this affects both desktop and mobile - though this would greatly benefit a mobile user where there is less space to fit in nodes/edges. I think this is covered by #6 |
https://1000i100.github.io/loopy/2/ is it better now ? You still cannot zoom with pinch gestures in the main canvas, but for other mobile disagreement, is-it fixed ? |
A great start! It seems that the toggle to close the menu doesn't quite work as expected, but other than that this would be a good improvement to merge in. The issue I found:
Expected: the menu to close. Maybe just some state not toggling properly? |
This wonderful little tool isn't that accessible from a mobile device. The elements are poorly sized and arranged, making them difficult to use.
At a cursory glance, this should only require CSS changes and a few minor tweaks to JS files for revealing/hiding elements.
The text was updated successfully, but these errors were encountered: