Comfy-Org / Comfy-Org/ComfyUI

[Bug] UI display priority

Open
#1,328 5 comments 1 reaction 0 assignees View on GitHub
Bug
Dominant language
Python
Stars
133k
Forks
15.7k
Avg merge
1d 10h
Merged PRs (30d)
153

Description

### Problem:
Currently, z-index of increments linearly, which places it over other boxes and eventually, even the menu.

Examples:
![chrome_BxMTQCsQx2](https://github.com/comfyanonymous/ComfyUI/assets/38716012/be0caa13-706b-4e04-bf85-84c07b8e8d2e)
![chrome_o0n9swzttq](https://github.com/comfyanonymous/ComfyUI/assets/38716012/49b688de-3c46-4afa-8537-5a399a85cf84)
![chrome_C874GtlsRS](https://github.com/comfyanonymous/ComfyUI/assets/38716012/4233dd12-171a-4d60-b92b-cdf18b6ce1d3)
![chrome_JVEW5OJXqC](https://github.com/comfyanonymous/ComfyUI/assets/38716012/a23a3c19-1bd2-4541-80de-28169c0fb2f3)
![chrome_vjgCFImDQb](https://github.com/comfyanonymous/ComfyUI/assets/38716012/e84eb4c2-d049-459a-b5a4-b06b8055576b)

### Expected:
Properties, pop ups, and menu should be above node and node contents.
![chrome_dWrvmagEMT](https://github.com/comfyanonymous/ComfyUI/assets/38716012/3517dc46-b1a3-45fe-b6aa-c2669019989e)

----------------------------------------------------------------------------------
For now, I've set custom user css locally to specifically move menu and properties window to 999 and 1000, but this doesn't seem like a good modular solution. Is there a reason for z-index to continually increment, or is this a bug? I can work on this issue if I get a conclusive answer, I'm just not sure if I'm overlooking something that necessitates said incrementations.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.