rio icon indicating copy to clipboard operation
rio copied to clipboard

Clearly Distinguish Sensitive from Insensitive Components

Open mad-moo opened this issue 1 year ago • 5 comments

Some components look virtually identical regardless of whether they're sensitive or not. Look through all components and make sure that's not the case.

For example, rio.Textinput: image

mad-moo avatar Nov 18 '24 21:11 mad-moo

I'll try this

Learnedprawn avatar Nov 19 '24 13:11 Learnedprawn

Beautiful. Thank you!

We're very active in discord if you need help with anything

mad-moo avatar Nov 19 '24 13:11 mad-moo

Okay I have gone through the project once, and now have a few questions. What exactly qualifies as sensitive and Insensitive?. And the way I am going to go about doing this task is:

  1. I created a new rio project called my-project.
  2. I am going to check each component by importing it into this project and find out the sensitive and the insensitive components.
  3. Change them so that its obvious which are sensitive and which are not. Any suggestions on this workflow? I will also ask the same on discord as you said you are active there too.

Learnedprawn avatar Nov 20 '24 11:11 Learnedprawn

I think you're misunderstanding something. It's not that some components are sensitive and some aren't, but rather, there are some components that have a is_sensitive parameter. For example, TextInput, Slider, Button, etc. The problem that needs to be solved is that a rio.TextInput(is_sensitive=True) looks too similar to a rio.TextInput(is_sensitive=False).

Aran-Fey avatar Nov 20 '24 12:11 Aran-Fey

Looking at the Dropdown demo, TextInput demo, and the explanation of the parameter ("Whether the dropdown / text input should respond to user input"), my feedback is threefold:

  1. The color scheme of the disabled Dropdown clearly indicates a difference from its enabled state, whereas the TextInput is almost identical - implement the Dropdown background coloring throughout
  2. The text of the disabled TextInput is much more visible than that of the disabled Dropbown - implement a suitable dark text color once (1) is changed
  3. The choice of "is_sensitive = True" to enable editing is confusing, so perhaps it would be better to change this to something like "is_sensitive = False" or "is_editable = True" or "is_readonly = True"

iwr-redmond avatar Dec 27 '24 23:12 iwr-redmond