From 3ba42bd91bcc70e6ab93da3cb4087d8829d850af Mon Sep 17 00:00:00 2001
From: pajlada <rasmus.karlsson@pajlada.com>
Date: Sun, 22 Jun 2025 16:04:06 +0200
Subject: [PATCH] fix: don't try to save emote popup bounds when we're quitting
 (#6292)

---
 CHANGELOG.md                       | 1 +
 src/widgets/dialogs/EmotePopup.cpp | 5 +++++
 2 files changed, 6 insertions(+)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index d34c34ab..24694bea 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -50,6 +50,7 @@
 - Dev: Added a `run-and-kill.sh` script to help debug crash-on-exit bugs. (#6188)
 - Dev: Refactored the `TimeoutStackStyle` enum into its own file. (#6216)
 - Dev: Refactored `Notebook`-related enums into their own file. (#6220)
+- Dev: Don't try to save emote popup bounds if we're quitting. (#6292)
 - Dev: Implemented customizable display names for enums. (#6238)
 - Dev: Refactored event API initialization away from Application and into TwitchIrcServer. (#6198)
 - Dev: Updated GoogleTest to v1.17.0. (#6180)
diff --git a/src/widgets/dialogs/EmotePopup.cpp b/src/widgets/dialogs/EmotePopup.cpp
index 798d41ec..404d8820 100644
--- a/src/widgets/dialogs/EmotePopup.cpp
+++ b/src/widgets/dialogs/EmotePopup.cpp
@@ -644,6 +644,11 @@ void EmotePopup::filterEmotes(const QString &searchText)
 
 void EmotePopup::saveBounds() const
 {
+    if (isAppAboutToQuit())
+    {
+        return;
+    }
+
     auto bounds = this->getBounds();
     if (!bounds.isNull())
     {