Complete rewrite of the tray icon implementation #6466

Hartmnt · 2024-06-12T20:30:44Z

FEAT(client): Fully rewrite tray icon implementation

The old tray icon implementation was very old and contained a lot of workaround for things that probably are no longer an issue.
Furthermore, the event loop was modified in a way such that it could end up in an infinite loop draining CPU time and rendering Mumble unusable.

Based on the new Qt5 implementation, this commit introduces a complete rewrite of the tray icon. The following things should be noted:

We assume the information in the Qt documentation [1] is valid. This means that all versions of Windows, all Linux window managers/compositors that implement the d-bus StatusNotifierItem specification, and all versions of macos support the functionality of QSystemTrayIcon and its notification system. That means we can drop the platform-specific code branches and handle messages directly with QSystemTrayIcon::sendMessage. This should for example also be true for recent versions of Gnome, which do not have an actual system tray, but implement the d-bus StatusNotifierItem specification. Therefore, we can actually merge and simplify the notification code for Windows and Unix*.
With regards to the bullet point above, we only limit the "hide to tray" functionality behind QSystemTrayIcon::isSystemTrayAvailable (because otherwise you would not get the Mumble window back without binding a shortcut first). Other code branches that were previously limited when isSystemTrayAvailable
returned false were removed. According to Qt, the QSystemTrayIcon code does not actually care if a system tray is available and will even retroactively add itself if a tray becomes available after the application was started.
On (X)Wayland, the minimize button in the window frame does not trigger a minimize change event. This means that users with such a system may only be able to "hide to tray" by 1) pressing the close button in the window frame and enabling "minimize instead of close" 2) clicking the tray icon or the tray icon hide action or 3) binding a shortcut to hide the window. This is either a bug or a deliberate decision by Qt or Wayland and we have no way to do anything
about that. (QTBUG-74310)
The "messageClicked" event is buggy in Qt on some platforms. That means that clicking the system notification spawned by Mumble via QSystemTrayIcon::sendMessage will (on some systems) never trigger anything especially not showing and activating the window. This is a long-standing bug in Qt (QTBUG-87329), but we have absolutely no way to work around this. The event is correctly hooked up in Mumble and if this is ever fixed in Qt, this will start working again automatically.
The tray icon has been redesigned according to state-of-the-art tray icon design guidelines [2]. Which basically just means: 1) d9a2d47 has been reverted to provide the user with a consistent menu 2) The main action of the tray icon (toggle show/hide) is the first entry in the context menu and the default action when the icon is clicked and 3) the TalkingUI toggle action was added. Actions for double and middle mouse clicks were removed as they might have contributed to infinite loops.
There is no way in Windows to show and activate a window that is not part of the current active process. If you have Mumble running in the background and receive a message, we can not raise the Window without
you clicking the Mumble taskbar item or tray icon yourself. This is deliberate by Microsoft and can and should not be circumvented. (Main window doesn't receive focus when Mumble-specific Windows notification is clicked #5701)
This also fixes the case where the Mumble MainWindow would disappear when pressing "OK" in the settings dialog. This happened because users would have "minimize to tray" and "minimize on close" enabled.

[1]
https://doc.qt.io/qt-5/qsystemtrayicon.html#details
https://doc.qt.io/qt-6/qsystemtrayicon.html#details

[2] https://learn.microsoft.com/en-us/windows/win32/uxguide/winenv-notification

FEAT(client): Implement tray icon highlighting

Previously, only "window highlighting" was supported by Mumble. However, when Mumble was minimized to tray,
there was no way to observe that highlighting.

As requested in #4584, this commit introduces highlighting to the tray icon. When a highlight message is received, the
tray icon will flash the "information icon" every two seconds until the MainWindow of Mumble receives focus again.

To enable this for a specific message type, the user needs to check the "highlight" box in the "Messages" configuration dialog.

FEAT(client): Add --hidden cli option to start Mumble hidden in tray

This commit introduces the "--hidden" cli option which prevents Mumble and the ConnectDialog to show up on startup.
This is especially useful for users who want to automate the startup process without human interaction.

Testing:

	Linux (X11 with XFCE)	Linux (XWayland KDE)	Windows	MacOS
Able to receive desktop notifications?	👍	👍	👍	👍
With minimize to tray enabled, does clicking the tray icon and/or clicking the first action of the context menu of the tray icon hide and show Mumble correctly?	👍	👍	👍	😐 (Left click opens context menu. Minimize instead of hide)
With minimize to tray and "ask on quit" dialog disabled, does pressing the close button in the title bar hide Mumble correctly?	👍	👍	👍	😐 (Minimize instead of hide)
With minimize to tray and "ask on quit" dialog enabled, does pressing the minimize button in the dialog hide Mumble correctly?	👍	👍	👍	❌ (Minimize only)
With minimize to tray enabled, does pressing the minimize button in the title bar hide Mumble correctly?	👍	❌	👍	❌ (Minimize only)
Is a maximized Mumble MainWindow still hiding and restoring correctly?	👍	👍	👍	😐 (Minimize instead of hide)
In Settings -> User Interface, is there a settings group box called "Tray Icon"?	👍	👍	👍	👍
Does pressing "OK" in the settings dialog not hide Mumble?	👍	👍	👍	👍
Receive a message for which "highlight" is enabled. Does the tray icon blink using the "i"-icon every two seconds until the MainWindow receives focus?	👍	👍	👍	👍
Does clicking a desktop notification restore and focus Mumble?	❌	❌	❌	👍
Does starting Mumble with `--hidden` actually work?	👍	👍	👍	❌ (Explicitly disabled on macOS)

Closes #1486
Closes #3028
Closes #3722
Closes #3879
Closes #3977
Closes #3999
Closes #4584
Closes #5012

src/mumble/Global.cpp

Krzmbrzl · 2024-06-15T07:56:59Z

(Potentially) relevant issues:

Hartmnt · 2024-06-15T08:22:42Z

@Krzmbrzl I considered #5261 but you can not really add text to the tray icon. So you would have to use the themed status icon you want to display and - at runtime - add some numbers to it, which seems... cumbersome... The other issues except #1574 are already referenced in the commit messages

Krzmbrzl · 2024-06-15T14:51:55Z

I just wanted these references to appear here in the GitHub UI explicitly to make sure everything is linked together correctly.

Snowknight26 · 2024-06-16T02:58:11Z

With https://dev.azure.com/Mumble-VoIP/Mumble/_build/results?buildId=8469&view=results, on Windows 10:

Able to receive desktop notifications?

Yes

With minimize to tray enabled, does clicking the tray icon and/or clicking the first action of the context menu of the tray icon hide and show Mumble correctly?

Yes

With minimize to tray and "ask on quit" dialog disabled, does pressing the close button in the title bar hide Mumble correctly?

Yes

With minimize to tray and "ask on quit" dialog enabled, does pressing the minimize button in the dialog hide Mumble correctly?

Yes

With minimize to tray enabled, does pressing the minimize button in the title bar hide Mumble correctly?

Yes

Is a maximized Mumble MainWindow still hiding and restoring correctly?

Yes

In Settings -> User Interface, is there a settings group box called "Tray"?

Yes ("Tray Icon")

Does pressing "OK" in the settings dialog not hide Mumble?

Yes

Receive a message for which "highlight" is enabled. Does the tray icon blink using the "i"-icon every two seconds until the MainWindow receives focus?

Not sure what "highlight" is enabled means in this context

Does clicking a desktop notification restore and focus Mumble?

Restore? Yes. Focus? No, taskbar icon keeps blinking but the Mumble window doesn't always become the the foreground one. In Main window doesn't receive focus when Mumble-specific Windows notification is clicked #5701 you wrote "Which means that I believe that Mumble should at least be correctly restored from minimization now", though this PR doesn't seem to make any changes to this.

Does starting Mumble with --hidden actually work?

Yes
Interestingly, having Mumble already started and then running mumble.exe --hidden again causes Mumble to appear in the task bar, blinking, but Mumble's main window does not appear. (Same behavior as 1.5.634 running mumble.exe again.) Starting Mumble for the first time using this command line argument, however seems to prevent Settings -> Network -> Reconnect to last server on startup from working.

Hartmnt · 2024-06-18T10:39:56Z

Not sure what "highlight" is enabled means in this context

There is a checkbox for "highlight" in the Settings -> Messages table view. If enabled, it should highlight the application in the task bar (some sort of blinking iirc) and as of this PR also change the tray icon. The effect should only apply, if Mumble has no focus and only lasts until the Window receives focus again.

though this PR doesn't seem to make any changes to this.

It does:

mumble/src/mumble/MainWindow.cpp

Lines 3631 to 3638 in e0cd306

    
           qDebug() << "Tray: Show raise window requested!"; 
        
           setWindowState(windowState() & ~Qt::WindowMinimized); 
        
           QTimer::singleShot(0, [this]() { 
        
           	show(); 
        
           	raise(); 
        
           	activateWindow(); 
        
           	setWindowState(windowState() | Qt::WindowActive); 
        
           });

It first restores the window from minimization and adds a timer to the raise and activate of the window.

Settings -> Network -> Reconnect to last server on startup

Good catch, I think this is a regression I should try to fix.

Snowknight26 · 2024-06-19T02:46:40Z

There is a checkbox for "highlight" in the Settings -> Messages table view.

That's what I was missing, thanks! Highlighting works as expected.

Unrelated to this PR but the first column of that table has some usability issues as the text is cut off, at least in Windows, making it hard to know what you're enabling highlighting (or any other option) for.

though this PR doesn't seem to make any changes to this.

It does:

mumble/src/mumble/MainWindow.cpp

Lines 3631 to 3638 in e0cd306

qDebug() << "Tray: Show raise window requested!";

setWindowState(windowState() & ~Qt::WindowMinimized);

QTimer::singleShot(0, [this]() {

show();

raise();

activateWindow();

setWindowState(windowState() | Qt::WindowActive);

});

It first restores the window from minimization and adds a timer to the raise and activate of the window.

Ahh sorry, what I meant was that during testing, I couldn't find any discernable difference in functionality between this version and 1.5.634.

Hartmnt · 2024-06-19T07:39:16Z

That's what I was missing, thanks! Highlighting works as expected.

Nice.

Ahh sorry, what I meant was that during testing, I couldn't find any discernable difference in functionality between this version and 1.5.634.

Ah ok. Well as stated in your original issue, it is sadly not possible for us to arbitrarily focus the Mumble MainWindow. But it is good to know that de-minimization works as expected.

Krzmbrzl · 2024-07-03T14:06:03Z

Is this what you had in mind? I guess it just feels weird to me because the signals are not emitted by the TrayIcon class, but all the calls now go through the signal slot mechanism, right?

Not quite - what you want here is to make e.g. the MainWindow have (new) signals that it emits. These are then wired up to the tray icon's slots. That makes things conventional again (signals of a class are only emitted by that class itself in reaction to an observed change).

all the calls now go through the signal slot mechanism, right?

I guess - I'm not sure how Qt handles class-external parts emitting a class's signals 😅

Hartmnt · 2024-07-03T14:15:09Z

the MainWindow have (new) signals that it emits

Ah. Ok. Well, that would work. I am just not sure, if that would make sense from a structural perspective. Because that would mean we would have tray icon specific signals in the MainWindow class just for that purpose?

I guess - I'm not sure how Qt handles class-external parts emitting a class's signals

Should make no difference as emit itself is a no-op.

Krzmbrzl · 2024-07-04T17:25:03Z

Ah. Ok. Well, that would work. I am just not sure, if that would make sense from a structural perspective. Because that would mean we would have tray icon specific signals in the MainWindow class just for that purpose?

Just don't frame them as tray-icon specific. The tray icon reacts to certain events. Make these events signals (with a name that only reflects the event, not something like makeTrayIconDoX). That should be possible, shouldn't it?

Should make no difference as emit itself is a no-op.

Yes, but Qt does some magic on its own when handling signals and slots (e.g. in order to implement the thread safety features)

Hartmnt · 2024-07-07T17:58:03Z

Just don't frame them as tray-icon specific. The tray icon reacts to certain events. Make these events signals (with a name that only reflects the event, not something like makeTrayIconDoX). That should be possible, shouldn't it?

I see. Yes, I guess that would work! Check out the latest changes to the experimental commit.

Krzmbrzl · 2024-07-08T06:39:00Z

Yes, that's what I had in mind 👍

src/mumble/widgets/TrayIcon.cpp

Hartmnt · 2024-07-13T08:16:03Z

@Krzmbrzl Anything left to do here? :)

src/mumble/Log.cpp

Krzmbrzl · 2024-08-04T19:42:49Z

src/mumble/Log.cpp

@@ -803,13 +805,32 @@ void Log::log(MsgType mt, const QString &console, const QString &terse, bool own
 		if (!(Global::get().mw->isActiveWindow() && Global::get().mw->qdwLog->isVisible())) {
 			// Message notification with window highlight
 			if (flags & Settings::LogHighlight) {
-				QApplication::alert(Global::get().mw);
+				Global::get().mw->highlightWindow();


should probably be done via a signal

Krzmbrzl · 2024-08-04T19:43:03Z

src/mumble/Log.cpp

+						msgIcon = QSystemTrayIcon::Information;
+						break;
+				}
+				Global::get().trayIcon->showMessage(msgName(mt), plain, msgIcon);


should probably be done via a signal

Krzmbrzl · 2024-08-04T19:44:29Z

src/mumble/MainWindow.cpp

+			bool wasMinimizedState = (windowStateEvent->oldState() & Qt::WindowMinimized);
+			bool isMinimizedState  = (windowState() & Qt::WindowMinimized);
+			if (!wasMinimizedState && isMinimizedState) {
+				Global::get().trayIcon->on_hideAction_triggered();


should probably be done via a signal

Krzmbrzl · 2024-08-04T19:44:46Z

src/mumble/MainWindow.cpp

-		} else {
-			Global::get().mw->show();
-		}
+		Global::get().trayIcon->toggleShowHide();


should probably be done via a signal

src/mumble/widgets/TrayIcon.cpp

Krzmbrzl · 2024-08-04T19:47:32Z

src/mumble/widgets/TrayIcon.cpp

+}
+
+void TrayIcon::on_icon_update() {
+	std::reference_wrapper< QIcon > newIcon = Global::get().mw->qiIcon;


why are you using a reference_wrapper for this? That class is usually only used when storing references as a member or in a container.

Krzmbrzl · 2024-08-04T19:50:20Z

Furthermore, I would like to ask you to link all issues that this PR (fully) addresses to the PR so that these get closed once we merge this PR (this can be done on the right side under the "Development" header).

The existing code for the tray icon was ancient, buggy and possibly entirely broken in more than one place. This commit removes all tray related code, such that it can be completely written from scratch.

The old tray icon implementation was very old and contained a lot of workaround for things that probably are no longer an issue. Furthermore, the event loop was modified in a way such that it could end up in an infinite loop draining CPU time and rendering Mumble unusable. Based on the new Qt5 implementation, this commit introduces a complete rewrite of the tray icon. The following things should be noted: * We assume the information in the Qt documentation [1] is valid. This means that all versions of Windows, all Linux window managers/compositors that implement the d-bus StatusNotifierItem specification, and all versions of macos support the functionality of QSystemTrayIcon and its notification system. That means we can drop the platform-specific code branches and handle messages directly with QSystemTrayIcon::sendMessage. This should for example also be true for recent versions of Gnome, which do not have an actual system tray, but implement the d-bus StatusNotifierItem specification. Therefore, we can actually merge and simplify the notification code for Windows and Unix*. * With regards to the bullet point above, we only limit the "hide to tray" functionality behind QSystemTrayIcon::isSystemTrayAvailable (because otherwise you would not get the Mumble window back without binding a shortcut first). Other code branches that were previously limited when isSystemTrayAvailable returned false were removed. According to Qt, the QSystemTrayIcon code does not actually care if a system tray is available and will even retroactively add itself if a tray becomes available after the application was started. * On (X)Wayland, the minimize button in the window frame does not trigger a minimize change event. This means that users with such a system may only be able to "hide to tray" by 1) pressing the close button in the window frame and enabling "minimize instead of close" 2) clicking the tray icon or the tray icon hide action or 3) binding a shortcut to hide the window. This is either a bug or a deliberate decision by Qt or Wayland and we have no way to do anything about that. (QTBUG-74310) * The "messageClicked" event is buggy in Qt on some platforms. That means that clicking the system notification spawned by Mumble via QSystemTrayIcon::sendMessage will (on some systems) never trigger anything especially not showing and activating the window. This is a long-standing bug in Qt (QTBUG-87329), but we have absolutely no way to work around this. The event is correctly hooked up in Mumble and if this is ever fixed in Qt, this will start working again automatically. * The tray icon has been redesigned according to state-of-the-art tray icon design guidelines [2]. Which basically just means: 1) d9a2d47 has been reverted to provide the user with a consistent menu 2) The main action of the tray icon (toggle show/hide) is the first entry in the context menu and the default action when the icon is clicked and 3) the TalkingUI toggle action was added. Actions for double and middle mouse clicks were removed as they might have contributed to infinite loops. * There is no way in Windows to show and activate a window that is not part of the current active process. If you have Mumble running in the background and receive a message, we can not raise the Window without you clicking the Mumble taskbar item or tray icon yourself. This is deliberate by Microsoft and can and should not be circumvented. (mumble-voip#5701) * This also fixes the case where the Mumble MainWindow would disappear when pressing "OK" in the settings dialog. This happened because users would have "minimize to tray" and "minimize on close" enabled. [1] https://doc.qt.io/qt-5/qsystemtrayicon.html#details https://doc.qt.io/qt-6/qsystemtrayicon.html#details [2] https://learn.microsoft.com/en-us/windows/win32/uxguide/winenv-notification Fixes mumble-voip#1486 Fixes mumble-voip#3028 Fixes mumble-voip#3722 Fixes mumble-voip#3977 Fixes mumble-voip#3999 Fixes mumble-voip#5012

Previously, only "window highlighting" was supported by Mumble. However, when Mumble was minimized to tray, there was no way to observe that highlighting. As requested in mumble-voip#4584, this commit introduces highlighting to the tray icon. When a highlight message is received, the tray icon will flash the "information icon" every two seconds until the MainWindow of Mumble receives focus again. To enable this for a specific message type, the user needs to check the "highlight" box in the "Messages" configuration dialog. Fixes mumble-voip#4584

This commit introduces the "--hidden" cli option which prevents Mumble and the ConnectDialog to show up on startup. This is especially useful for users who want to automate the startup process without human interaction. Fixes mumble-voip#3879

Krzmbrzl

(Not sure if this is waiting on a review so I just reviewed to be sure :D)

Otherwise the unresolved comments from above remain valid.

Krzmbrzl · 2024-10-03T16:31:28Z

src/mumble/widgets/TrayIcon.cpp

@@ -0,0 +1,203 @@
+// Copyright 2024 The Mumble Developers. All rights reserved.


The date should be removed now that we have removed it from all other copyright headers on master

Krzmbrzl · 2024-10-03T16:32:12Z

src/mumble/widgets/TrayIcon.cpp

+#include "../ClientUser.h"
+#include "../MainWindow.h"
+#include "../Global.h"


Couldn't do the include "normally" as these files should be compiled with the regular include path? 🤔

Having explicit include paths is always a bit fishy 👀

Krzmbrzl · 2024-10-03T16:34:22Z

src/mumble/widgets/TrayIcon.cpp

+void TrayIcon::on_icon_update() {
+	std::reference_wrapper< QIcon > newIcon = Global::get().mw->qiIcon;
+
+	ClientUser *p = ClientUser::get(Global::get().uiSession);


Suggested change

ClientUser *p = ClientUser::get(Global::get().uiSession);

const ClientUser *p = ClientUser::get(Global::get().uiSession);

Hartmnt · 2024-10-03T16:48:17Z

(Not sure if this is waiting on a review so I just reviewed to be sure :D)

Just have to get to this PR again at some point :) I want do the fixes for 1.5 first

Krzmbrzl reviewed Jun 13, 2024

View reviewed changes

src/mumble/Global.cpp Outdated Show resolved Hide resolved

Hartmnt force-pushed the fix_tray branch 8 times, most recently from 38aace8 to 3af870a Compare June 14, 2024 15:52

This was referenced Jun 14, 2024

Main window doesn't receive focus when Mumble-specific Windows notification is clicked #5701

Closed

'Hide in tray when minimized' feature doesn't work on Wayland #5012

Closed

Hartmnt force-pushed the fix_tray branch from 3af870a to e0cd306 Compare June 14, 2024 22:17

Krzmbrzl mentioned this pull request Jun 15, 2024

Show number of participants in currently joined channel in tray icon #5261

Closed

Hartmnt force-pushed the fix_tray branch 10 times, most recently from 953ecff to fdbb8cb Compare June 23, 2024 19:06

Hartmnt force-pushed the fix_tray branch from 087dad3 to 1aa99c4 Compare July 7, 2024 17:49

Krzmbrzl reviewed Jul 8, 2024

View reviewed changes

src/mumble/widgets/TrayIcon.cpp Outdated Show resolved Hide resolved

Hartmnt force-pushed the fix_tray branch 4 times, most recently from 3fc78a6 to 55ea907 Compare July 8, 2024 13:02

Hartmnt requested a review from Krzmbrzl July 8, 2024 13:04

Krzmbrzl reviewed Aug 4, 2024

View reviewed changes

Hartmnt linked an issue Aug 4, 2024 that may be closed by this pull request

Cannot minimize Mumble with bHideInTray=true on Ubuntu 16.10/Unity 7.5.0 #3028

Open

Hartmnt added 2 commits September 14, 2024 11:17

REFAC(client): Replace 0 with nullptr in Global.cpp

5a63f67

REFAC(client): Remove old tray icon code

00f35bd

The existing code for the tray icon was ancient, buggy and possibly entirely broken in more than one place. This commit removes all tray related code, such that it can be completely written from scratch.

Hartmnt force-pushed the fix_tray branch 4 times, most recently from 65848df to d5380c6 Compare September 16, 2024 14:04

Hartmnt added 4 commits September 16, 2024 14:06

TRANSLATION: Update translation files

06a0a0a

Hartmnt force-pushed the fix_tray branch from d5380c6 to 06a0a0a Compare September 16, 2024 14:07

Krzmbrzl reviewed Oct 3, 2024

View reviewed changes

Krzmbrzl mentioned this pull request Oct 3, 2024

Mumble crashing chaning from default white to dark theme #6434

Closed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Complete rewrite of the tray icon implementation #6466

Complete rewrite of the tray icon implementation #6466

Hartmnt commented Jun 12, 2024 •

edited

Loading

Krzmbrzl commented Jun 15, 2024

Hartmnt commented Jun 15, 2024

Krzmbrzl commented Jun 15, 2024

Snowknight26 commented Jun 16, 2024 •

edited

Loading

Hartmnt commented Jun 18, 2024

Snowknight26 commented Jun 19, 2024

Hartmnt commented Jun 19, 2024

Krzmbrzl commented Jul 3, 2024

Hartmnt commented Jul 3, 2024

Krzmbrzl commented Jul 4, 2024

Hartmnt commented Jul 7, 2024

Krzmbrzl commented Jul 8, 2024

Hartmnt commented Jul 13, 2024

Krzmbrzl Aug 4, 2024

Krzmbrzl Aug 4, 2024

Krzmbrzl Aug 4, 2024

Krzmbrzl Aug 4, 2024

Krzmbrzl Aug 4, 2024

Krzmbrzl commented Aug 4, 2024

Krzmbrzl left a comment

Krzmbrzl Oct 3, 2024

Krzmbrzl Oct 3, 2024

Krzmbrzl Oct 3, 2024

Hartmnt commented Oct 3, 2024

		@@ -0,0 +1,203 @@
		// Copyright 2024 The Mumble Developers. All rights reserved.

	ClientUser *p = ClientUser::get(Global::get().uiSession);
	const ClientUser *p = ClientUser::get(Global::get().uiSession);

Complete rewrite of the tray icon implementation #6466

Are you sure you want to change the base?

Complete rewrite of the tray icon implementation #6466

Conversation

Hartmnt commented Jun 12, 2024 • edited Loading

Krzmbrzl commented Jun 15, 2024

Hartmnt commented Jun 15, 2024

Krzmbrzl commented Jun 15, 2024

Snowknight26 commented Jun 16, 2024 • edited Loading

Hartmnt commented Jun 18, 2024

Snowknight26 commented Jun 19, 2024

Hartmnt commented Jun 19, 2024

Krzmbrzl commented Jul 3, 2024

Hartmnt commented Jul 3, 2024

Krzmbrzl commented Jul 4, 2024

Hartmnt commented Jul 7, 2024

Krzmbrzl commented Jul 8, 2024

Hartmnt commented Jul 13, 2024

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Krzmbrzl commented Aug 4, 2024

Krzmbrzl left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Hartmnt commented Oct 3, 2024

Hartmnt commented Jun 12, 2024 •

edited

Loading

Snowknight26 commented Jun 16, 2024 •

edited

Loading