comfyanonymous
New Menu too Uniform with Nodes
New menu theme is just blending with the nodes, so when you have entire screen full of nodes, I tend to actually have to look for my menu. :P There should be contrast from the rest of the UI elements as a control panel / menu.
Additionally, menu has no UX
- No hover events
- No click events
- No tooltips (buttons that can destroy a session should have tool tip explanations of their function, like Refresh, and prossibly confirmation)
- Mute colors that are not definable so not a good help for color blind, or those with lack of light perception making similar tones blend together.
- Extension of 4, some font is almost same grey as background, and people like my father (who likes SD) didn't even know the text was there to do batches.
Not only do I lose the menu as looking like a node itself, but when zoomed in with lots of nodes, because it's color is identical to a node, can literally blend right into the nodes because it has no border, or box shadow to emphasize it as a floating element above the node plane.
the menu's drop shadow is right/bottom, so moving it to the upper left corner makes it more obvious. I do think the shadows can use a few more pixels width though. Also help for the "active node" glow.
the menu's drop shadow is right/bottom, so moving it to the upper left corner makes it more obvious. I do think the shadows can use a few more pixels width though. Also help for the "active node" glow.
Probably should also not be a directional shadow since it is a positional menu you can put anywhere the user wants. But the menu in general should have a differentiating theme from the graph too. It's you main triggering point. Shouldn't have to look for it. Should be obvious.
replacing the the "Queue Prompt" button with a big orange "Generate" would differentiate the menu
replacing the the "Queue Prompt" button with a big orange "Generate" would differentiate the menu
Yeah, or just change the color theme of the button in general. I don't know about "generate" cause it is literally a queue button.
Being able to toggle it to contrast with the current browser light/dark setting, maybe? eg, dark theme when the browser is light.
Being able to toggle it to contrast with the current browser light/dark setting, maybe? eg, dark theme when the browser is light.
Ooh, yeah that would be nice. A prefers-color-scheme or whatever is called env var to determine colors
@WASasquatch Did you try the new Beta UI? Is this still an issue? https://github.com/comfyanonymous/ComfyUI/pull/3112
I haven't tried the new beta UI. Hadn't heard about it. O.o
But it is still an issue with original UI. The menu has no bordering and shares the same color as nodes.
On Wed, Jul 3, 2024, 3:20 PM Robin Huang @.***> wrote:
@WASasquatch https://github.com/WASasquatch Did you try the new Beta UI? Is this still an issue? #3112 https://github.com/comfyanonymous/ComfyUI/pull/3112
— Reply to this email directly, view it on GitHub https://github.com/comfyanonymous/ComfyUI/issues/379#issuecomment-2207402121, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAIZEZJCF2UJKIPL6SQVPLLZKR2LBAVCNFSM6AAAAABKKMY5J2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMBXGQYDEMJSGE . You are receiving this because you were mentioned.Message ID: @.***>
This issue has been transferred to the frontend repository: https://github.com/Comfy-Org/ComfyUI_frontend/issues/6452
Please continue the discussion there.
