InvokeAI icon indicating copy to clipboard operation
InvokeAI copied to clipboard

[bug]: Input fields can NO longer be removed

Open Neosettler opened this issue 8 months ago • 4 comments

Is there an existing issue for this problem?

  • [x] I have searched the existing issues

Operating system

Windows

GPU vendor

Nvidia (CUDA)

GPU model

3090

GPU VRAM

No response

Version number

5.9

Browser

brave

Python dependencies

No response

What happened

Once an input field have been added to the builder form using the + button. There is no - button to remove it.

What you expected to happen

The + button should become - button as soon as an input has been added to the form builder. (as seen in the previous versions)

How to reproduce the problem

No response

Additional context

No response

Discord username

No response

Neosettler avatar Apr 03 '25 01:04 Neosettler

You can remove it with the X button in the builder.

The button does not turn in to a minus sign bc you can add a field multiple times to the form. In that case, which version of the field do you want to remove? Maybe it's a mistake to let you add a field multiple times, and we should change that behavior.

psychedelicious avatar Apr 03 '25 01:04 psychedelicious

I can't see where having the same field twice would be necessary. For what it's worth, I much liked the "minus sign" as it was very handy.

Neosettler avatar Apr 03 '25 04:04 Neosettler

And/or to improve the nodes visual feedback. I'd suggest to differentiate the fields that are linked to the form builder. Text color or bold font would work just fine.

Neosettler avatar Apr 03 '25 04:04 Neosettler

@psychedelicious from our previous conversation,

"I was thinking, isExposed feedback would work just fine but if you'd like to go the extra mile to bring back the delete feature, I'd suggest "remove all" if it has duplicates or even better, keep isExposed true if it has another, then, user can press delete again. it would make sense to go with "last in" "first out" logic"."

Neosettler avatar Apr 08 '25 00:04 Neosettler