Use a direct Lighten instead of >1f Color4 values

This commit is contained in:
Alden Wu
2022-11-22 00:31:22 -08:00
parent 4084a2b066
commit a2b505f4c0

View File

@ -61,6 +61,8 @@ namespace osu.Game.Overlays.Chat
private OsuSpriteText username = null!; private OsuSpriteText username = null!;
private Drawable usernameColouredDrawable = null!;
private Container? highlight; private Container? highlight;
private readonly Bindable<bool> prefer24HourTime = new Bindable<bool>(); private readonly Bindable<bool> prefer24HourTime = new Bindable<bool>();
@ -89,6 +91,10 @@ namespace osu.Game.Overlays.Chat
? Color4Extensions.FromHex(message.Sender.Colour) ? Color4Extensions.FromHex(message.Sender.Colour)
: username_colours[message.Sender.Id % username_colours.Length]; : username_colours[message.Sender.Id % username_colours.Length];
// this can be either the username sprite text or a container
// around it depending on which branch is taken in createUsername()
var usernameDrawable = createUsername();
InternalChild = new GridContainer InternalChild = new GridContainer
{ {
RelativeSizeAxes = Axes.X, RelativeSizeAxes = Axes.X,
@ -113,13 +119,13 @@ namespace osu.Game.Overlays.Chat
Colour = colourProvider?.Background1 ?? Colour4.White, Colour = colourProvider?.Background1 ?? Colour4.White,
AlwaysPresent = true, AlwaysPresent = true,
}, },
new MessageSender(message.Sender) new MessageSender(message.Sender, usernameColouredDrawable)
{ {
Width = UsernameWidth, Width = UsernameWidth,
AutoSizeAxes = Axes.Y, AutoSizeAxes = Axes.Y,
Origin = Anchor.TopRight, Origin = Anchor.TopRight,
Anchor = Anchor.TopRight, Anchor = Anchor.TopRight,
Child = createUsername(), Child = usernameDrawable,
Margin = new MarginPadding { Horizontal = Spacing }, Margin = new MarginPadding { Horizontal = Spacing },
}, },
ContentFlow = new LinkFlowContainer(t => ContentFlow = new LinkFlowContainer(t =>
@ -198,7 +204,6 @@ namespace osu.Game.Overlays.Chat
username = new OsuSpriteText username = new OsuSpriteText
{ {
Shadow = false, Shadow = false,
Colour = senderHasColour ? colours.ChatBlue : usernameColour,
Truncate = true, Truncate = true,
EllipsisString = "…", EllipsisString = "…",
Font = OsuFont.GetFont(size: TextSize, weight: FontWeight.Bold, italics: true), Font = OsuFont.GetFont(size: TextSize, weight: FontWeight.Bold, italics: true),
@ -208,7 +213,13 @@ namespace osu.Game.Overlays.Chat
}; };
if (!senderHasColour) if (!senderHasColour)
{
usernameColouredDrawable = username;
usernameColouredDrawable.Colour = usernameColour;
return username; return username;
}
username.Colour = colours.ChatBlue;
// Background effect // Background effect
return new Container return new Container
@ -233,7 +244,7 @@ namespace osu.Game.Overlays.Chat
CornerRadius = 4, CornerRadius = 4,
Children = new Drawable[] Children = new Drawable[]
{ {
new Box usernameColouredDrawable = new Box
{ {
RelativeSizeAxes = Axes.Both, RelativeSizeAxes = Axes.Both,
Colour = usernameColour, Colour = usernameColour,
@ -252,15 +263,19 @@ namespace osu.Game.Overlays.Chat
private class MessageSender : OsuClickableContainer, IHasContextMenu private class MessageSender : OsuClickableContainer, IHasContextMenu
{ {
private readonly APIUser sender; private readonly APIUser sender;
private readonly Drawable colouredDrawable;
private readonly Color4 defaultColour;
private Action startChatAction = null!; private Action startChatAction = null!;
[Resolved] [Resolved]
private IAPIProvider api { get; set; } = null!; private IAPIProvider api { get; set; } = null!;
public MessageSender(APIUser sender) public MessageSender(APIUser sender, Drawable colouredDrawable)
{ {
this.sender = sender; this.sender = sender;
this.colouredDrawable = colouredDrawable;
defaultColour = colouredDrawable.Colour;
} }
[BackgroundDependencyLoader] [BackgroundDependencyLoader]
@ -295,7 +310,7 @@ namespace osu.Game.Overlays.Chat
protected override bool OnHover(HoverEvent e) protected override bool OnHover(HoverEvent e)
{ {
this.FadeColour(new Color4(1.4f, 1.4f, 1.4f, 1), 150, Easing.OutQuint); colouredDrawable.FadeColour(defaultColour.Lighten(0.4f), 150, Easing.OutQuint);
return base.OnHover(e); return base.OnHover(e);
} }
@ -304,7 +319,7 @@ namespace osu.Game.Overlays.Chat
{ {
base.OnHoverLost(e); base.OnHoverLost(e);
this.FadeColour(Color4.White, 250, Easing.OutQuint); colouredDrawable.FadeColour(defaultColour, 250, Easing.OutQuint);
} }
} }