search

chrisgoringe / cg-image-picker

227 stars 15 forks source link

HUD status location conflicts with altered layout from Cute-Comfy #58

Closed DrJKL closed 9 months ago

DrJKL commented 9 months ago

Cute ComfyUI moves some of the floating Comfy HUD into a frame with the logo/save/load in the upper left.

The Image Chooser HUD puts the running node information in the same location as fixed.

This looks a little messy (can't read either the status or the logo), but my main issue is sometimes being unable to click the Save button, depending on the size of the window and the length of the name of the running node.

In terms of possible fixes, I'm not sure how to resolve the combinations, i.e. Just cg-image-picker vs cg-image-picker + cute-comfy without special casing (something I generally try to avoid). I think moving the fixed top offset down by an assumed header's height might be fine, but could be awkward looking in vanilla Comfy, I haven't tested it.

(A complete side-note, it'd be nice to add a class to the div/span to indicate their origin/intent)

DrJKL commented 9 months ago

Just Cute

image

Cute + Image Chooser

image

DrJKL commented 9 months ago

top: 60px;

image

chrisgoringe commented 9 months ago

Maybe I just add an option for the hud (on/off at least, perhaps position).

chrisgoringe commented 9 months ago

Added settings option to move the HUD (or hide it).