From 5521e696e50d31a64efd96739dc2a42b7cd2628f Mon Sep 17 00:00:00 2001 From: Christian Bastian <80225746+cdb-boop@users.noreply.github.com> Date: Mon, 9 Sep 2024 03:01:09 -0400 Subject: [PATCH] Fixed modal `z-index` for old and beta menu. --- web/model-manager.css | 6 +++++- web/model-manager.js | 11 +++++++++-- 2 files changed, 14 insertions(+), 3 deletions(-) diff --git a/web/model-manager.css b/web/model-manager.css index 787758f..913f286 100644 --- a/web/model-manager.css +++ b/web/model-manager.css @@ -10,7 +10,7 @@ position: fixed; overflow: hidden; width: 100%; - z-index: 99; /*needs to be below dialog modal*/ + z-index: 2000; /*needs to be below the dialog modal element*/ /*override comfy-modal settings*/ border-radius: 0; @@ -23,6 +23,10 @@ touch-action: manipulation; } +.model-manager .model-manager-dialog { + z-index: 2001; /*needs to be above the model manager element*/ +} + .model-manager .comfy-modal-content { width: 100%; gap: 16px; diff --git a/web/model-manager.js b/web/model-manager.js index 2913314..e452285 100644 --- a/web/model-manager.js +++ b/web/model-manager.js @@ -164,9 +164,9 @@ async function tryOpenModelUrl(modelSearchPath) { try { window.open(modelUrl, '_blank').focus(); } - catch (exception) { + catch (exception) { // browser or ad-blocker blocking opening new window - app.ui.dialog.show($el("span", + modelManagerDialog.show($el("span", [ $el("p", { style: { color: "var(--input-text)" }, @@ -4951,12 +4951,19 @@ class ModelManager extends ComfyDialog { /** @type {ModelManager | undefined} */ let instance; +/** @type {ComfyDialog | undefined} */ +let modelManagerDialog; + /** * @returns {ModelManager} */ function getInstance() { if (!instance) { instance = new ModelManager(); + + modelManagerDialog = new ComfyDialog(); + modelManagerDialog.element.classList.add("model-manager-dialog"); + instance.element.appendChild(modelManagerDialog.element); } return instance; }