YKariyado/LG

More issues...

julovi opened this issue · 3 comments

julovi commented

  • Why is important that the x and z axes are labeled? I think it looks better without "(x)", likewise without "Time (z)"

  • The load button still opens buttons, it's not going directly to the file browser. I'm not sure how to express it better: Load -> file browser. The File browser window opens in a directory where the presets are located. It is the default directory, but the user can choose a different directory. This would be the same behavior for saving. Think about Word: when you press the open menu, does it give you buttons to the previous files you edited? no! it opens a file browser...

  • Moving the BPM slider also moves the model

  • The BPM is wrong when "sequential" is not set, it should go much faster, check,

  • the head seems to be not centered, can you check? it looks as if it's closer to the top of the blue axis

  • I like the rectangle showing the current location in the sequential mode, but may be a bit narrower would be better?

  • I can't load blinker one and 2

  • Running the presets is broken, it doesn't change

  • The head rotation is still funny, if I use the up/down arrows, it doesn't really rotate as expected.

  • In the sandpile model (maybe in both?), I think is more natural to count from 1. So, (4,4,4) should be at the fourth dot starting from each axis, right now it's the fifth.

  • Setting a different coordinate and number of grains is not working, check. (i.e, set a coordinate and # of grains, let it run and try to set a different number of grains or coordinate...)

  • Non-sequential doesn't really work on the sandpile. I'm not sure why... check

  • Add a better description of the project in the Github page.

YKariyado commented

I could load both of blinker in ma environment so idk what happened but ima bring to fix it :/

julovi commented

please checy my comment on how the load button needs to work. let me know if you don't understand it

arevaloarboled commented
  • i can delete it, just as a reference.
  • i change it to open search the files, but we show it to you and you told us not like that. Adjusted as is requested in this issue.
  • i check it, there is a rule that if the mouse raycast with a UI element, it will not interact with the behind elements.
  • i think is because if it is not sequential is being triggered just only the first beat (out of 4). should be every beat?
  • fixed.
  • too narrower will be difficult to perceive, i put it a little bit narrower.
  • solved.
  • ??
  • you mean invert the axis of rotation?
    Until here everything solved. @YKariyado , can you help me with the rest defined here?