diff --git a/osu.Game/Online/Chat/Channel.cs b/osu.Game/Online/Chat/Channel.cs index 59d0cd8cc1..92e93d4a08 100644 --- a/osu.Game/Online/Chat/Channel.cs +++ b/osu.Game/Online/Chat/Channel.cs @@ -30,6 +30,8 @@ namespace osu.Game.Online.Chat //internal bool Joined; + public const int MAX_HISTORY = 100; + [JsonConstructor] public Channel() { @@ -47,13 +49,9 @@ namespace osu.Game.Online.Chat private void purgeOldMessages() { - const int max_history = 50; - int messageCount = Messages.Count; - if (messageCount > 50) - { - Messages.RemoveRange(0, messageCount - max_history); - } + if (messageCount > MAX_HISTORY) + Messages.RemoveRange(0, messageCount - MAX_HISTORY); } } } diff --git a/osu.Game/Online/Chat/Display/ChannelDisplay.cs b/osu.Game/Online/Chat/Display/ChannelDisplay.cs index ad7db24a55..4c6bc4af80 100644 --- a/osu.Game/Online/Chat/Display/ChannelDisplay.cs +++ b/osu.Game/Online/Chat/Display/ChannelDisplay.cs @@ -67,13 +67,13 @@ namespace osu.Game.Online.Chat.Display { if (!IsLoaded) return; - var displayMessages = newMessages.Skip(Math.Max(0, newMessages.Count() - 20)); + var displayMessages = newMessages.Skip(Math.Max(0, newMessages.Count() - Channel.MAX_HISTORY)); - //up to last 20 messages + //up to last Channel.MAX_HISTORY messages foreach (Message m in displayMessages) flow.Add(new ChatLine(m)); - while (flow.Children.Count() > 20) + while (flow.Children.Count() > Channel.MAX_HISTORY) flow.Remove(flow.Children.First()); } }