ncase / loopy

A tool for thinking in systems
http://ncase.me/loopy/
Creative Commons Zero v1.0 Universal
1.62k stars 212 forks source link

Make the UI mobile-friendly #17

Open neverstew opened 5 years ago

neverstew commented 5 years ago

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.

1000i100 commented 4 years ago

Do you feel the play mode isn't mobile friendly ? Or only the edit mode ?

neverstew commented 4 years ago

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:

  1. The sidebar is fixed.
  2. You cannot zoom with pinch gestures in the main canvas.

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

1000i100 commented 4 years ago

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 ?

neverstew commented 4 years ago

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:

  1. Draw some nodes
  2. Select a node (the menu appears)
  3. Select the arrow to close the menu

Expected: the menu to close. Actual: the menu stays open. Another click of the arrow closes it.

Maybe just some state not toggling properly?