From 5e66a2bb5acc6e3ba146af7323f1d36eba342a96 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Rafa=C5=82=20Dzi=C4=99giel?= <rafostar.github@gmail.com>
Date: Mon, 22 Feb 2021 11:27:20 +0100
Subject: [PATCH] Wait for top revealer transition to finish before changing it

Changing transition in middle can have dire consequences, so change only when not in transition.
---
 src/revealers.js | 16 +++++++++++++++-
 1 file changed, 15 insertions(+), 1 deletion(-)

diff --git a/src/revealers.js b/src/revealers.js
index 09e6b62e..bbdc7860 100644
--- a/src/revealers.js
+++ b/src/revealers.js
@@ -48,6 +48,7 @@ class ClapperRevealerTop extends CustomRevealer
             valign: Gtk.Align.START,
         });
         this.revealerName = 'top';
+        this._requestedTransition = this.transition_type;
 
         const initTime = GLib.DateTime.new_now_local().format('%X');
         this.timeFormat = (initTime.length > 8)
@@ -123,6 +124,8 @@ class ClapperRevealerTop extends CustomRevealer
         revealerBox.append(this.revealerGrid);
 
         this.set_child(revealerBox);
+
+        this.connect('notify::child-revealed', this._onTopRevealed.bind(this));
     }
 
     set title(text)
@@ -171,9 +174,20 @@ class ClapperRevealerTop extends CustomRevealer
 
         this.headerBar.extraButtonsBox.visible = !isFullscreen;
 
-        this.transition_type = (isTvMode)
+        this._requestedTransition = (isTvMode)
             ? Gtk.RevealerTransitionType.SLIDE_DOWN
             : Gtk.RevealerTransitionType.CROSSFADE;
+
+        /* Changing transition in middle can have dire consequences,
+           so change only when not in transition */
+        if(this.reveal_child === this.child_revealed)
+            this.transition_type = this._requestedTransition;
+    }
+
+    _onTopRevealed()
+    {
+        if(this.transition_type !== this._requestedTransition)
+            this.transition_type = this._requestedTransition;
     }
 });