Skip to content

Commit

Permalink
Don't animate admin monitor popup on page load, properly remove inter…
Browse files Browse the repository at this point in the history
…actable UI elements (jenkinsci#8954)

* followup for jenkinsci#8941, don't animate on page load

PR jenkinsci#8941 caused a regression that the hiding animation was played on
page load.
This change ensures that the hiding animation is only applied when the
widget was visible before

* scale to 0

(cherry picked from commit e5fd912)
  • Loading branch information
mawinter69 authored and NotMyFault committed Feb 13, 2024
1 parent 6bc8a8a commit 4089150
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -50,8 +50,12 @@
box-shadow: var(--dropdown-box-shadow);
border-radius: 15px;
opacity: 0;
animation: hide-am-list 300ms ease-in 1 normal;
z-index: 0;
transform: scale(0);
}

.am-container.am-hidden div.am-list {
animation: hide-am-list 300ms ease-in 1 normal;
}

.am-container.visible div.am-list {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -38,6 +38,7 @@
var popupContent = responseText;
amList.innerHTML = popupContent;
amMonitorRoot.classList.add("visible");
amMonitorRoot.classList.remove("am-hidden");
document.addEventListener("click", onClose);
document.addEventListener("keydown", onEscClose);

Expand All @@ -50,6 +51,9 @@
}

function close() {
if (amMonitorRoot.classList.contains("visible")) {
amMonitorRoot.classList.add("am-hidden");
}
amMonitorRoot.classList.remove("visible");
document.removeEventListener("click", onClose);
document.removeEventListener("keydown", onEscClose);
Expand Down

0 comments on commit 4089150

Please sign in to comment.