actual icon indicating copy to clipboard operation
actual copied to clipboard

[Feedback]: Toggle Switch

Open carkom opened this issue 1 year ago • 3 comments

I've noticed that there's a few places in the app where we use a static "toggle" menu item but have no way to tell current state of that element (eg show/hide categories on budget page).

Current (both hidden and shown are the same): image

A couple possible options here: iconology:

Hidden Shown
OR also using dynamic text
Hidden Shown

Dynamic Text (no icons):

Hidden Shown
Toggle Switch (status follows toggle):
Hidden Shown

carkom avatar Dec 20 '23 10:12 carkom

Maybe there's other ideas or better implementation of any of the above options?

For me, I find the iconology options a bit difficult to understand. The Dynamic text is used often in other places of the app, I don't always like that the actual state of the element in question has to be inferred by the user (example voice in my head: "okay I have an option to show hidden categories, I guess that means that they are currently not shown"). So this leaves the toggle switch which has my vote. It's simple and quick to understand and use.

I'd love to hear everyone's thoughts and preferences. Please feel free to add other options or tweak the ones I have above.

carkom avatar Dec 20 '23 11:12 carkom

I agree completely with your analysis.

Iconology contains too little visual differentiation, I at first thought you'd made an error in posting and put the same image in twice as I was only skimming images.

Dynamic text kinda works, but also it means there's effectively twice the UI options (and the mental processing you mentioned).

Toggle switch keeps the one UI element, is visually distinctive, is very instantly recognisable as a toggle(no mental processing). Also I like the splash of colour.

Teprifer avatar Dec 20 '23 11:12 Teprifer

+1 for toggle switch!

jsehnoutka avatar Dec 22 '23 00:12 jsehnoutka

@carkom Do you need this to still be open? If we want to add toggles places maybe we should open issues for those.

youngcw avatar Feb 26 '24 21:02 youngcw

Implemented in #2174

carkom avatar Feb 26 '24 22:02 carkom