From 017d38af3d0f8af13155cf049e27c5371fd6f3bf Mon Sep 17 00:00:00 2001
From: Dean Herbert <pe@ppy.sh>
Date: Tue, 28 Jan 2025 21:29:17 +0900
Subject: [PATCH] Change friend online notifications' icon and colours

The previous choices made it seem like potentially destructive actions
were being performed. I've gone with neutral colours and more suiting
icons to attempt to avoid this.

---

Addresses concerns in
https://github.com/ppy/osu/discussions/31621#discussioncomment-11948377.

I chose this design even though it wasn't the #1 most popular because I
personally feel that using green/red doesn't work great for these.
---
 osu.Game/Online/FriendPresenceNotifier.cs | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/osu.Game/Online/FriendPresenceNotifier.cs b/osu.Game/Online/FriendPresenceNotifier.cs
index 75b487384a..bc2bf344b0 100644
--- a/osu.Game/Online/FriendPresenceNotifier.cs
+++ b/osu.Game/Online/FriendPresenceNotifier.cs
@@ -171,9 +171,9 @@ namespace osu.Game.Online
             {
                 Transient = true,
                 IsImportant = false,
-                Icon = FontAwesome.Solid.UserPlus,
+                Icon = FontAwesome.Solid.User,
                 Text = $"Online: {string.Join(@", ", onlineAlertQueue.Select(u => u.Username))}",
-                IconColour = colours.Green,
+                IconColour = colours.GrayD,
                 Activated = () =>
                 {
                     if (singleUser != null)
@@ -208,9 +208,9 @@ namespace osu.Game.Online
             {
                 Transient = true,
                 IsImportant = false,
-                Icon = FontAwesome.Solid.UserMinus,
+                Icon = FontAwesome.Solid.UserSlash,
                 Text = $"Offline: {string.Join(@", ", offlineAlertQueue.Select(u => u.Username))}",
-                IconColour = colours.Red
+                IconColour = colours.Gray3
             });
 
             offlineAlertQueue.Clear();