Skip to content

Commit

Permalink
Don't show loading tooltip for now
Browse files Browse the repository at this point in the history
Should probably be replaced with a loading spinner in the future, don't
really like "loading" tooltips.
  • Loading branch information
peppy committed Nov 26, 2021
1 parent d5d1a4c commit b8a4775
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions osu.Game/Screens/OnlinePlay/DrawableRoomPlaylistItem.cs
Original file line number Diff line number Diff line change
Expand Up @@ -147,7 +147,8 @@ private void refresh()
if (showItemOwner)
{
ownerAvatar.Show();
userLookupCache.GetUserAsync(Item.OwnerID).ContinueWith(u => Schedule(() => ownerAvatar.User = u.Result), TaskContinuationOptions.OnlyOnRanToCompletion);
userLookupCache.GetUserAsync(Item.OwnerID)
.ContinueWith(u => Schedule(() => ownerAvatar.User = u.Result), TaskContinuationOptions.OnlyOnRanToCompletion);
}

difficultyIconContainer.Child = new DifficultyIcon(Item.Beatmap.Value, ruleset.Value, requiredMods, performBackgroundDifficultyLookup: false) { Size = new Vector2(ICON_HEIGHT) };
Expand Down Expand Up @@ -458,7 +459,7 @@ public OwnerAvatar()
});
}

public LocalisableString TooltipText => User == null ? "loading user..." : $"queued by {User.Username}";
public LocalisableString TooltipText => User == null ? string.Empty : $"queued by {User.Username}";

private class TooltipArea : Component, IHasTooltip
{
Expand Down

0 comments on commit b8a4775

Please sign in to comment.