enhancement: updates character window and quest log

- simplifies extra buffs details into a button with tooltip
- updates strings for extra buffs button with tooltip.
- Weylon's review -> rather updates tooltip on hover.

Author: Arufonsu

Intersect.Client.Core/Interface/Game/Character/CharacterWindow.cs

@@ -9,8 +9,6 @@
 using Intersect.Enums;
 using Intersect.Framework.Core.GameObjects.Items;
 using Intersect.Framework.Core.GameObjects.PlayerClass;
-using Intersect.GameObjects;
-using Intersect.Network.Packets.Server;
 
 namespace Intersect.Client.Interface.Game.Character;
 
@@ -71,40 +69,24 @@ public partial class CharacterWindow
     public int Y;
 
     //Extra Buffs
+    Button _detailsButton;
+    
     ClassDescriptor mPlayer;
 
-    Label mHpRegen;
-
     long HpRegenAmount;
 
-    Label mManaRegen;
-
     long ManaRegenAmount;
 
-    Label mLifeSteal;
-
     int LifeStealAmount = 0;
 
-    Label mAttackSpeed;
-
-    Label mExtraExp;
-
     int ExtraExpAmount = 0;
 
-    Label mLuck;
-
     int LuckAmount = 0;
 
-    Label mTenacity;
-
     int TenacityAmount = 0;
 
-    Label mCooldownReduction;
-
     int CooldownAmount = 0;
 
-    Label mManaSteal;
-
     int ManaStealAmount = 0;
 
     //Init
@@ -168,18 +150,12 @@ public CharacterWindow(Canvas gameCanvas)
             Items[i].Setup();
         }
 
-        var extraBuffsLabel = new Label(mCharacterWindow, "ExtraBuffsLabel");
-        extraBuffsLabel.SetText(Strings.Character.ExtraBuffs);
-
-        mHpRegen = new Label(mCharacterWindow, "HpRegen");
-        mManaRegen = new Label(mCharacterWindow, "ManaRegen");
-        mLifeSteal = new Label(mCharacterWindow, "Lifesteal");
-        mAttackSpeed = new Label(mCharacterWindow, "AttackSpeed");
-        mExtraExp = new Label(mCharacterWindow, "ExtraExp");
-        mLuck = new Label(mCharacterWindow, "Luck");
-        mTenacity = new Label(mCharacterWindow, "Tenacity");
-        mCooldownReduction = new Label(mCharacterWindow, "CooldownReduction");
-        mManaSteal = new Label(mCharacterWindow, "Manasteal");
+        _detailsButton = new Button(mCharacterWindow, nameof(_detailsButton))
+        {
+            Text = Strings.Character.ExtraBuffDetails,
+        };
+        _detailsButton.HoverEnter += UpdateExtraBuffTooltip; // Update Tooltip on hover.
+        UpdateExtraBuffTooltip(null, null); // Initial tooltip update.
 
         mCharacterWindow.LoadJsonUi(GameContentManager.UI.InGame, Graphics.Renderer.GetResolutionString());
     }
@@ -411,29 +387,16 @@ public void UpdateExtraBuffs()
         mPlayer = ClassDescriptor.Get(Globals.Me?.Class ?? Guid.Empty);
 
         //Getting HP and Mana Regen
-        if (mPlayer != null)
-        {
-            HpRegenAmount = mPlayer.VitalRegen[0];
-            mHpRegen.SetText(Strings.Character.HealthRegen.ToString(HpRegenAmount));
-            ManaRegenAmount = mPlayer.VitalRegen[1];
-            mManaRegen.SetText(Strings.Character.ManaRegen.ToString(ManaRegenAmount));
-        }
+        HpRegenAmount = mPlayer?.VitalRegen[(int)Vital.Health] ?? 0;
+        ManaRegenAmount = mPlayer?.VitalRegen[(int)Vital.Mana] ?? 0;
 
-        CooldownAmount = 0;
+        //Reset all values
         LifeStealAmount = 0;
         TenacityAmount = 0;
         LuckAmount = 0;
         ExtraExpAmount = 0;
         ManaStealAmount = 0;
-
-        mLifeSteal.SetText(Strings.Character.Lifesteal.ToString(0));
-        mExtraExp.SetText(Strings.Character.ExtraExp.ToString(0));
-        mLuck.SetText(Strings.Character.Luck.ToString(0));
-        mTenacity.SetText(Strings.Character.Tenacity.ToString(0));
-        mCooldownReduction.SetText(Strings.Character.CooldownReduction.ToString(0));
-        mManaSteal.SetText(Strings.Character.Manasteal.ToString(0));
-
-        mAttackSpeed.SetText(Strings.Character.AttackSpeed.ToString(Globals.Me.CalculateAttackTime() / 1000f));
+        CooldownAmount = 0;
     }
 
     /// <summary>
@@ -449,20 +412,18 @@ public void UpdateExtraBuffs(Guid itemId)
             return;
         }
 
-        //Getting HP and Mana Regen
-        if (item.VitalsRegen[0] != 0)
+        //Getting HP and Mana Regen from items
+        if (item.VitalsRegen[(int)Vital.Health] != 0)
         {
-            HpRegenAmount += item.VitalsRegen[0];
-            mHpRegen?.SetText(Strings.Character.HealthRegen.ToString(HpRegenAmount));
+            HpRegenAmount += item.VitalsRegen[(int)Vital.Health];
         }
 
-        if (item.VitalsRegen[1] != 0)
+        if (item.VitalsRegen[(int)Vital.Mana] != 0)
         {
-            ManaRegenAmount += item.VitalsRegen[1];
-            mManaRegen?.SetText(Strings.Character.ManaRegen.ToString(ManaRegenAmount));
+            ManaRegenAmount += item.VitalsRegen[(int)Vital.Mana];
         }
 
-        //Getting extra buffs
+        //Getting extra buffs from items
         if (item.Effects.Find(effect => effect.Type != ItemEffect.None && effect.Percentage > 0) != default)
         {
             foreach(var effect in item.Effects)
@@ -476,39 +437,42 @@ public void UpdateExtraBuffs(Guid itemId)
                 {
                     case ItemEffect.CooldownReduction:
                         CooldownAmount += effect.Percentage;
-                        mCooldownReduction?.SetText(Strings.Character.CooldownReduction.ToString(CooldownAmount));
-
                         break;
                     case ItemEffect.Lifesteal:
                         LifeStealAmount += effect.Percentage;
-                        mLifeSteal?.SetText(Strings.Character.Lifesteal.ToString(LifeStealAmount));
-
                         break;
                     case ItemEffect.Tenacity:
                         TenacityAmount += effect.Percentage;
-                        mTenacity?.SetText(Strings.Character.Tenacity.ToString(TenacityAmount));
-
                         break;
                     case ItemEffect.Luck:
                         LuckAmount += effect.Percentage;
-                        mLuck?.SetText(Strings.Character.Luck.ToString(LuckAmount));
-
                         break;
                     case ItemEffect.EXP:
                         ExtraExpAmount += effect.Percentage;
-                        mExtraExp?.SetText(Strings.Character.ExtraExp.ToString(ExtraExpAmount));
-
                         break;
                     case ItemEffect.Manasteal:
                         ManaStealAmount += effect.Percentage;
-                        mManaSteal?.SetText(Strings.Character.Manasteal.ToString(ManaStealAmount));
-
                         break;
                 }
             }
         }
     }
 
+    public void UpdateExtraBuffTooltip(Base? sender, EventArgs? arguments)
+    {
+        var tooltip = new System.Text.StringBuilder();
+        tooltip.AppendLine(Strings.Character.HealthRegen.ToString(HpRegenAmount));
+        tooltip.AppendLine(Strings.Character.ManaRegen.ToString(ManaRegenAmount));
+        tooltip.AppendLine(Strings.Character.Lifesteal.ToString(LifeStealAmount));
+        tooltip.AppendLine(Strings.Character.AttackSpeed.ToString(Globals.Me?.CalculateAttackTime() / 1000f));
+        tooltip.AppendLine(Strings.Character.ExtraExp.ToString(ExtraExpAmount));
+        tooltip.AppendLine(Strings.Character.Luck.ToString(LuckAmount));
+        tooltip.AppendLine(Strings.Character.Tenacity.ToString(TenacityAmount));
+        tooltip.AppendLine(Strings.Character.CooldownReduction.ToString(CooldownAmount));
+        tooltip.AppendLine(Strings.Character.Manasteal.ToString(ManaStealAmount));
+        _detailsButton.SetToolTipText(tooltip.ToString());
+    }
+
     /// <summary>
     /// Show the window
     /// </summary>

Intersect.Client.Core/Localization/Strings.cs

@@ -637,7 +637,7 @@ public partial struct Character
         public static LocalizedString Equipment = @"Equipment:";
 
         [JsonProperty(NullValueHandling = NullValueHandling.Ignore)]
-        public static LocalizedString ExtraBuffs = @"Extra Buffs";
+        public static LocalizedString ExtraBuffDetails = @"Details";
 
         [JsonProperty(NullValueHandling = NullValueHandling.Ignore)]
         public static LocalizedString ExtraExp = @"Bonus EXP: {00}%";