[UX Review] Fine-tuning of the multiple values field

[xianli123]

Describe the bug

In the new UI, there are some fields that can be filled by multiple values as shown below. Taking the Valid redirect URIs as an example, when there are multiple values, each value should be able to delete, so the remove icon should be clickable for all fields.

Version

Admin new UI

Expected behavior

No response

Actual behavior

No response

How to Reproduce?

No response

Anything else?

No response

[edewit]

Right I guess it should be the other way around only the first field can't be deleted

[edewit]

That way you can always return to the initial state:

[xianli123]

Hi @edewit @jonkoops I think it will be better if each value can be removed. What do you think?

[jonkoops]

Can you log a new issue for it Kun?

[edewit]

but @xianli123 when I remove the last field there will be nothing left? Is that what you want like this:

[xianli123]

Hi @edewit No. We should allow users to delete one specific field(including the last one), not all fields.

[edewit]

ahh okay so you mean only disable the first field if the first field is the only field

[xianli123]

Yes. If there are multiple values, the first value can also be removed.

…

On Mon, Aug 15, 2022 at 8:31 PM Erik Jan de Wit ***@***.***> wrote: ahh okay so you mean only disable the first field if the first field is the only field — Reply to this email directly, view it on GitHub <#18175>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AM3O372G5BRFPOQJQRRZR7DVZI2DFANCNFSM54ULBPEA> . You are receiving this because you were mentioned.Message ID: ***@***.***>

-- Xiankun Li Interaction Designer (UXD Team) <https://red.ht/sig>