h1or4dme8/osu-lazer
1
0
Fork 0
mirror of https://github.com/ppy/osu.git synced 2024-11-13 22:07:25 +08:00
Code Issues Releases Wiki Activity

Improve message highlight handling in DrawableChannel

Browse Source
This commit is contained in:
Salman Ahmed 2022-03-10 23:06:58 +03:00
parent a31611bdec
commit d07e3101ea
1 changed files with 25 additions and 21 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

46
osu.Game/Overlays/Chat/DrawableChannel.cs
View File

@ -87,29 +87,31 @@ namespace osu.Game.Overlays.Chat
base.LoadComplete(); base.LoadComplete();
highlightedMessage = Channel.HighlightedMessage.GetBoundCopy(); highlightedMessage = Channel.HighlightedMessage.GetBoundCopy();
highlightedMessage.BindValueChanged(m => highlightedMessage.BindValueChanged(_ => processMessageHighlighting(), true);
{
if (m.NewValue == null)
return;
// schedule highlight to ensure it performs after any pending "newMessagesArrived" calls.
// also schedule after children to ensure the scroll flow is updated with any new chat lines.
ScheduleAfterChildren(() =>
{
var chatLine = chatLines.SingleOrDefault(c => c.Message.Equals(m.NewValue));
if (chatLine != null)
{
float center = scroll.GetChildPosInContent(chatLine, chatLine.DrawSize / 2) - scroll.DisplayableContent / 2;
scroll.ScrollTo(Math.Clamp(center, 0, scroll.ScrollableExtent));
chatLine.Highlight();
}
highlightedMessage.Value = null;
});
}, true);
} }
/// <summary>
/// Processes any pending message in <see cref="highlightedMessage"/>.
/// </summary>
/// <remarks>
/// <see cref="CompositeDrawable.ScheduleAfterChildren"/> is for ensuring the scroll flow has updated with any new chat lines.
/// </remarks>
private void processMessageHighlighting() => ScheduleAfterChildren(() =>
{
if (highlightedMessage.Value == null)
return;
var chatLine = chatLines.SingleOrDefault(c => c.Message.Equals(highlightedMessage.Value));
if (chatLine == null)
return;
float center = scroll.GetChildPosInContent(chatLine, chatLine.DrawSize / 2) - scroll.DisplayableContent / 2;
scroll.ScrollTo(Math.Clamp(center, 0, scroll.ScrollableExtent));
chatLine.Highlight();
highlightedMessage.Value = null;
});
protected override void Dispose(bool isDisposing) protected override void Dispose(bool isDisposing)
{ {
base.Dispose(isDisposing); base.Dispose(isDisposing);
@ -179,6 +181,8 @@ namespace osu.Game.Overlays.Chat
// to avoid making the container think the user has scrolled back up and unwantedly disable auto-scrolling. // to avoid making the container think the user has scrolled back up and unwantedly disable auto-scrolling.
if (newMessages.Any(m => m is LocalMessage)) if (newMessages.Any(m => m is LocalMessage))
scroll.ScrollToEnd(); scroll.ScrollToEnd();
processMessageHighlighting();
}); });
private void pendingMessageResolved(Message existing, Message updated) => Schedule(() => private void pendingMessageResolved(Message existing, Message updated) => Schedule(() =>