From 1d38e5bd33ad8764ae3b16b468a967430174f3e3 Mon Sep 17 00:00:00 2001
From: Dean Herbert <pe@ppy.sh>
Date: Thu, 12 May 2022 13:58:10 +0900
Subject: [PATCH] Add notes about weird local handling in `ChannelListSelector`

---
 osu.Game/Overlays/Chat/ChannelList/ChannelListSelector.cs | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/osu.Game/Overlays/Chat/ChannelList/ChannelListSelector.cs b/osu.Game/Overlays/Chat/ChannelList/ChannelListSelector.cs
index 7beef7b16c37..a07aad20412a 100644
--- a/osu.Game/Overlays/Chat/ChannelList/ChannelListSelector.cs
+++ b/osu.Game/Overlays/Chat/ChannelList/ChannelListSelector.cs
@@ -73,6 +73,8 @@ protected override void LoadComplete()
 
             currentChannel.BindValueChanged(channel =>
             {
+                // This logic should be handled by the chat overlay rather than this component.
+                // Selected state should be moved to an abstract class and shared with ChannelListItem.
                 if (channel.NewValue == null)
                 {
                     text.FadeColour(colourProvider.Content1, 300, Easing.OutQuint);