Nachtzuster / BirdNET-Pi

A realtime acoustic bird classification system for the Raspberry Pi 5, 4B 3B+ 0W2 and more. Built on the TFLite version of BirdNET.
https://birdnetpi.com
Other
169 stars 24 forks source link

UI : Convert checkbox to toggleswitch #209

Closed alexbelgium closed 1 month ago

alexbelgium commented 1 month ago

Not very useful but prettier :-) Tested on windows and ios

Before :

image

After : image image image image

bdeschut commented 1 month ago

Personal opinion: for me it wouldn't be clear if the toggle is on or off, given that the color or the text doesn't change (I refer to the two screenshots you provided). Perhaps add some color? image

alexbelgium commented 1 month ago

Thanks for the feedback, indeed makes total sense ! As I haven't modified the main css files to make it simpler, it would need to cover both light and dark themes. Would such a layout be ok :

image image

bdeschut commented 1 month ago

It is definitely an improvement. Not sure if difficult, but for the dark theme (and the light theme as well), this might also be an option: image

alexbelgium commented 1 month ago

Thanks for the suggestions. I'm not a fan of text on small elements like that - it would in my opinion lead to clutter especially on smaller screens. However I do understand that the difference of light vs grey shades could be not super explicit.

Here is an update :

image image image image

Alternative : checkmarks

image
alexbelgium commented 1 month ago

What is preferrable : "check" or "ON"

bdeschut commented 1 month ago

Both are clear for me, I slightly prefer the check sign more but both are excellent options

alexbelgium commented 1 month ago

Thanks for your feedbacks and proposals

Nachtzuster commented 1 month ago

Not very useful but prettier :-) Tested on windows and ios

quite :smile: