bradmartin333 / SEYR

Simple Entropy Yield Routine
MIT License
1 stars 0 forks source link

Fresh Eyes Feedback #122

Closed riversn closed 2 years ago

riversn commented 2 years ago

Had some completely fresh eyes trying to use SEYR.

Biggest thing:

riversn commented 2 years ago

Finally got them to add the latest seyr/gridmaker to the next version of xferprint, it's going live on monday... so will actually get to use those quality of life improvements i've been waiting on

riversn commented 2 years ago

"apply angle/deskew" should just be next to the angle numbers... new users don't understand it or know it was there. and they didn't understand that changing the angle was even worthwhile.

"Auto Calc Angle" button should just be to the right of the angle values instead of under tools.

"Find Angle" idk the good name

bradmartin333 commented 2 years ago

"apply angle/deskew" should just be next to the angle numbers... new users don't understand it or know it was there. and they didn't understand that changing the angle was even worthwhile.

"Auto Calc Angle" button should just be to the right of the angle values instead of under tools.

"Find Angle" idk the good name

Added as #123

bradmartin333 commented 2 years ago
  • The yellow 'no images' page wasn't clear enough, user didn't know what it meant - probably need some text on the picture saying "no images are being saved, are you sure" or something?

I have some text in the bar at the bottom:

image

But I will make the image with an arrow that points to that in #124

bradmartin333 commented 2 years ago
  • Clicking inside of the box should highlight it on the right screen.

I think this one is done 👍

  • Corners should be draggable (maybe only top left/bottom right). Click, hold, and dragging should be able to move the box position. or clicking the edge of the box should be able to drag and move it too.

I can do this in https://github.com/bradmartin333/LI4 but it is not really practical for SEYR (you would always be frustrated with edge cases due to scaling)

bradmartin333 commented 2 years ago
  • "Img" unit inspect boxes shouldn't be showing up in the viewer because we don't really care about the img pass/fail rate and seeing the larger img box fail a unit when the LEDs are actually there is very confusing for a new user

Opened in #125

bradmartin333 commented 2 years ago

Had some completely fresh eyes trying to use SEYR.

That's great to hear! Also, great feedback - I can do most of these easily, but the click and drag stuff won't perform well. I am planning to make LI4 the successor to SEYR and those sorts of things will all be built in as well as database integration. On another note, the database testing with SEYR went well. If it becomes imperative I can revisit that - otherwise, I will crawl along with LI4.

riversn commented 2 years ago

sounds good, we aren't desperate for anything database wise because the system we have now just works (why change if it just works)?

bradmartin333 commented 2 years ago

love to hear it - just finished making those changes I took from this post