From fd97c2268ab2cf8e839919d60a4f8c887a95feb9 Mon Sep 17 00:00:00 2001 From: Lucas Nicodemus Date: Sat, 12 Jun 2021 13:22:56 -0700 Subject: [PATCH] Send success messages as Color.LightGreen Basically Color.Green looks ugly and Color.LightGreen looks less ugly. I could see changing the default to something like pure green, but I think this might cause people to be like "yo why is this so bright." Now arguably this should be configurable but I'm not going to revolutionize the world by abandoning far more important work to allow more artful messages. I just think that Color.Green is ugly enough that I can't tolerate it. --- CHANGELOG.md | 1 + TShockAPI/TSPlayer.cs | 2 +- 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index d1d6ac53..6efd6085 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -18,6 +18,7 @@ This is the rolling changelog for TShock for Terraria. Use past tense when addin * Added `/god` as an alias for `/godmode` to be more consistent with other server mods. (@hakusaro) * Fixed ridiculous typo in `Amethyst Gemtree` text. (@hakusaro) * Fixed `CTRL + C` / interactive console interrupt not safely shutting down the server. Now, interrupts will cause a safe shutdown (saving the world and disconnecting all players before fully shutting down). Previously, interrupts caused an unsafe shutdown (not saving the world). (@hakusaro) +* Changed "success message" color to `Color.LimeGreen` instead of `Color.Green`. `Color.Green` looks ugly. `Color.LimeGreen` looks less ugly but isn't as offensively bright as pure green. (@hakusaro) ## TShock 4.5.4 * Fixed ridiculous typo in `GetDataHandlers` which caused TShock to read the wrong field in the packet for `usingBiomeTorches`. (@hakusaro, @Arthri) diff --git a/TShockAPI/TSPlayer.cs b/TShockAPI/TSPlayer.cs index aeaf17b3..f4b25752 100644 --- a/TShockAPI/TSPlayer.cs +++ b/TShockAPI/TSPlayer.cs @@ -1379,7 +1379,7 @@ namespace TShockAPI /// The message. public virtual void SendSuccessMessage(string msg) { - SendMessage(msg, Color.Green); + SendMessage(msg, Color.LimeGreen); } ///