terrakok / Compose-Multiplatform-Wizard

Compose Multiplatform Wizard
https://terrakok.github.io/Compose-Multiplatform-Wizard/
MIT License
389 stars 25 forks source link

Minor UI improvements #12

Closed cjbrooks12 closed 1 year ago

cjbrooks12 commented 1 year ago

Adds title attributes to icons, and "Enabled/Disabled" text to Target buttons for clarity.

terrakok commented 1 year ago

I'm ready to approve part with the top menu titles. But I don't like texts under target buttons. To understand how they work one click is enough, I think

cjbrooks12 commented 1 year ago

I really think the target buttons need something other than color to indicate whether they're selected or not. If you have to click a button to understand what it does, that's a problem in terms of usability. And personally, I was still unsure of whether colored meant selected or unselected even after clicking on it; it wasn't until I actually downloaded the project that I knew for sure which targets were selected.

How would you feel if I were to try some other patterns to emphasize whether a button is selected or not? For example, like shown in this article.

image

terrakok commented 1 year ago

Yes, let's try that one

terrakok commented 1 year ago
image