From 1be6c54be28df2be005fb2b7e40572d7713695dd Mon Sep 17 00:00:00 2001 From: Andy Wang Date: Thu, 26 Mar 2020 22:45:03 +0000 Subject: [PATCH] Fix dropup menu being covered --- src/web/waiters/RecipeWaiter.mjs | 15 --------------- 1 file changed, 15 deletions(-) diff --git a/src/web/waiters/RecipeWaiter.mjs b/src/web/waiters/RecipeWaiter.mjs index 4f8290f4..ba0e7b11 100755 --- a/src/web/waiters/RecipeWaiter.mjs +++ b/src/web/waiters/RecipeWaiter.mjs @@ -51,7 +51,6 @@ class RecipeWaiter { } }.bind(this), onSort: function(evt) { - this.updateZIndices(); if (evt.from.id === "rec-list") { document.dispatchEvent(this.manager.statechange); } @@ -150,19 +149,6 @@ class RecipeWaiter { } - /** - * Sets the z-index property on each operation to make sure that operations higher in the list - * have a higher index, meaning dropdowns are not hidden underneath subsequent operations. - */ - updateZIndices() { - const operations = document.getElementById("rec-list").children; - for (let i = 0; i < operations.length; i++) { - const operation = operations[i]; - operation.style.zIndex = 100 + operations.length - i; - } - } - - /** * Handler for favourite dragover events. * If the element being dragged is an operation, displays a visual cue so that the user knows it can @@ -480,7 +466,6 @@ class RecipeWaiter { log.debug(`'${e.target.querySelector(".op-title").textContent}' added to recipe`); this.triggerArgEvents(e.target); - this.updateZIndices(); window.dispatchEvent(this.manager.statechange); }