fix(skills-overview): hide xp details in tooltip after reaching 99

osrs-tracker · Feb 28, 2024 · 0031a53 · 0031a53
1 parent 3a1ed8a
commit 0031a53
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/src/app/common/components/player/player-skill.component.ts b/src/app/common/components/player/player-skill.component.ts
@@ -26,14 +26,14 @@ import { TooltipComponent } from '../general/tooltip/tooltip.component';
       <div class="flex justify-between gap-4">
         <div>
           <div>{{ skill()?.name }} XP:</div>
-          @if (skill()?.name !== SkillEnum.Overall) {
+          @if (showXpDetails) {
             <div>Next Level at:</div>
             <div>Remaining XP:</div>
           }
         </div>
         <div class="text-right">
           <div>{{ skill()?.xp | number }}</div>
-          @if (skill()?.name !== SkillEnum.Overall) {
+          @if (showXpDetails) {
             <div>{{ xpForNextLevel | number }}</div>
             <div>{{ xpToNextLevel | number }}</div>
           }
@@ -56,4 +56,8 @@ export class PlayerSkillWidgetComponent {
   get xpForNextLevel(): number {
     return calculateXPForSkillLevel((this.skill()?.level ?? 1) + 1);
   }
+
+  get showXpDetails(): boolean {
+    return this.skill()?.name !== SkillEnum.Overall && this.skill()?.level !== 99;
+  }
 }