Can I convince someone to rename `Allowed Images` to `Limit Allowed Images`

I ll try to bring it up in the UI UX sync meeting slightly smiling face

To be discussed in <https github com rancher sandbox rancher desktop discussions 3699|Allowed Image Patterns label Discussion 3699 rancher sandbox rancher desktop>

Can I convince someone to rename `Allowed Images` to `Limit rancher-users #rancher-desktop

#rancher-desktop

Title

# rancher-desktop

quick-keyboard-83126

01/03/2023, 7:36 PM

Can I convince someone to rename

Allowed Images

Limit Allowed Images

fast-garage-66093

01/03/2023, 7:47 PM

Probably not. I called it "Image Allow List" and it was renamed to "Allowed Images", and everything is being changed to match that (internally names as still mixed).

Anyways, I don't see why "Limit Allowed Images" would be better

quick-keyboard-83126

01/03/2023, 7:48 PM

The reason is that the

Enable

thing really doesn't make sense as-is

w/ it unchecked, it implies that now images are allowed

whereas what it means is that the feature to limit allowed images isn't enabled

fast-garage-66093

01/03/2023, 7:49 PM

I'll try to bring it up in the UI/UX sync meeting... 🙂

👍 1

quick-keyboard-83126

01/03/2023, 7:49 PM

If instead of checkbox, you used a radio w/ two states one being "all" and the other being "selected"

... that'd also work if they're really attached to the title

fast-garage-66093

01/04/2023, 9:34 PM

To be discussed in Allowed Image Patterns label · Discussion #3699 · rancher-sandbox/rancher-desktop

🙌 1

8 Views