raphaelgoulart / ya_inputdisplay

Yet another input display for Clone Hero / YARG
16 stars 3 forks source link

[Suggestions] Various ideas relating to "clean look" #21

Closed SkyJumper409x closed 2 months ago

SkyJumper409x commented 2 months ago

I recently heard feedback on the IPS, upon asking why the person preferred the IPS invisible in their Stream layout. They replied that "It just looks cleaner" and they "don't even know what IPS means". This got me thinking, and in my opinion, a config option to hide the IPS would be nice to have for people who are not particularly interested in the metric.

I am thinking of moving the InputBtns up if both the IPS Object and Hamburger button are hidden, which would allow for a more "clean" look. I am not sure what to do with the Hamburger button then, possibly move it to the side?

Additionally, what about all the empty width of the window? What about starting the input display with the window cropped to fit the InputBtns. Still keeping all the height of course, only affecting the width of the window. Maybe even save whatever aspect ratio the window currently has upon closing, so it persists between program runs?

raphaelgoulart commented 2 months ago

IPS being togglable makes sense. It means "Inputs per second" (though you probably know it), just a metric to "how fast you're going" for whoever cares (mariko's input display had it so I just yoinked the idea from that)

I am thinking of moving the InputBtns up (...)

not strictly needed imo, the user can just move the capture up on OBS themselves if they want. Just making IPS togglable should be good, it'd avoid stuff moving without the user explicitly telling it to, and would also avoid having to reposition the hamburger button, which could confuse the user as well.

Regarding window size, your suggestions are good!

raphaelgoulart commented 2 months ago

Fixed by PR https://github.com/raphaelgoulart/ya_inputdisplay/pull/23