From 75d834dcc3f1637f70712e4128ab1fe202bda270 Mon Sep 17 00:00:00 2001
From: Anders Kaseorg <anders@zulip.com>
Date: Fri, 6 Sep 2024 14:38:40 -0700
Subject: [PATCH] styles: Remove ineffective animation of display.
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

It’s not possible to animate or transition the `display` property, at
least until `transition-behavior: allow-discrete` lands in all
browsers.  We already take care of applying `display: none` in a
JavaScript setTimeout (see alert_popup.ts, hide_error in
ui_report.ts).

Signed-off-by: Anders Kaseorg <anders@zulip.com>
---
 web/styles/alerts.css | 2 --
 1 file changed, 2 deletions(-)

diff --git a/web/styles/alerts.css b/web/styles/alerts.css
index ca3df6ae90..2bec56e098 100644
--- a/web/styles/alerts.css
+++ b/web/styles/alerts.css
@@ -214,7 +214,6 @@ $alert-error-red: hsl(0deg 80% 40%);
 /* animation section */
 @keyframes fadeIn {
     0% {
-        display: block;
         opacity: 0;
         transform: translateY(-100px);
     }
@@ -232,7 +231,6 @@ $alert-error-red: hsl(0deg 80% 40%);
     }
 
     100% {
-        display: none;
         opacity: 0;
         transform: translateY(-100px);
     }