[Feature Request]: Ability to HIDE fields from Preview display

Comfy-Org / ComfyUI_frontend

Official front-end implementation of ComfyUI

https://www.comfy.org/

GNU General Public License v3.0

576 stars 100 forks source link

[Feature Request]: Ability to HIDE fields from Preview display #309

Closed Amorano closed 1 month ago

Amorano commented 2 months ago

Is there an existing issue for this?

[X] I have searched the existing issues and checked the recent builds/commits

What would your feature do ?

It would add a meta attr to the field define to hide it from display in the preview box.

This would also go along to resolve issue #302

Proposed workflow

"VERTEX_PROGRAM": ("STRING", {"default": GLSLShader.PROG_VERTEX}) ==> "VERTEX_PROGRAM": ("STRING", {"default": GLSLShader.PROG_VERTEX, "hide": True})

Additional information

No response

huchenlei commented 2 months ago

Have you considered putting the metadata into hidden?

https://github.com/Comfy-Org/ComfyUI_frontend/blob/1b96c16cca55ac2f51be492eb1d0b79b9f1e0706/src/types/apiTypes.ts#L259-L261

Amorano commented 2 months ago

Have you considered putting the metadata into hidden?

https://github.com/Comfy-Org/ComfyUI_frontend/blob/1b96c16cca55ac2f51be492eb1d0b79b9f1e0706/src/types/apiTypes.ts#L259-L261

Hidden is neato, but it literally removes the node from being seen, anywhere. The fields are still used as display on the node they just do not need to show their defaults as obnoxious as they do.

Amorano commented 2 months ago

hide from preview; still used.