bradmartin333 / SEYR

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

More user feedback for click grid functions #126

Closed riversn closed 2 years ago

riversn commented 2 years ago

when clicking to 'move' or place the grid box on the left side, it is correctly centering around the courser, but it's just barely misaligned. (cntl + G) It's almost as if it's being placed to the center of the mouse, not the point of the mouse. Maybe backing off the offset by 1 px would do it.

image

bradmartin333 commented 2 years ago

before, it used the top left corner and didn't have the issue. now, it uses the "center" but the center has to be a pixel and, depending on your scaling, it could seem like it is one pixel off for edge cases

bradmartin333 commented 2 years ago

tried floor, ceiling, +1 and -1 and they all have the same edge case issue (there just aren't that many pixels in the tile)

riversn commented 2 years ago

It's actually perfect on the right screen, but on the left grid it's slightly off... idk, maybe it's just me

bradmartin333 commented 2 years ago

left = grid, right = feature

the left is perfect and the right is slightly off, correct?

if so, then it is not just you

riversn commented 2 years ago

No, it's opposite, the inspection viewer on the right is correct.

The left side grid array is slightly off

bradmartin333 commented 2 years ago

https://user-images.githubusercontent.com/19335151/184418757-5d8b8d2c-449f-4b63-88d1-238959954553.mp4

left feels good, right is understandably off (but that is what I want to make "feel" better without actually changing the outcome)

bradmartin333 commented 2 years ago

I did a good amount of testing here (I even dabbled with click and drag/resize functions), but I can't seem to make this "feel" better without redoing the entire form.

riversn commented 2 years ago

Well... it's good enough. If it can't be easily fixed then oh well