Merge pull request #424 from mbeps/development

BUG FIX: Minimise skill button not being shown when skills table expanded
mbeps · Jul 31, 2024 · 15ab5e1 · 15ab5e1
2 parents 1550a27 + 9864b12
commit 15ab5e1
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/components/Skills/CategorySkillDisplay.tsx b/components/Skills/CategorySkillDisplay.tsx
@@ -33,7 +33,10 @@ const CategorySkillDisplay: React.FC<CategorySkillDisplayProps> = ({
   let skillCount: number = 0;
   let groupCount: number = 0;
 
-  function filterSkills(skills: SkillDatabaseKeys[], onlyMain: boolean) {
+  function filterSkills(
+    skills: SkillDatabaseKeys[],
+    onlyMain: boolean
+  ): SkillDatabaseKeys[] {
     return skills.filter((skillKey) =>
       onlyMain ? skillDatabaseMap[skillKey]?.isMainSkill : true
     );
@@ -98,7 +101,7 @@ const CategorySkillDisplay: React.FC<CategorySkillDisplayProps> = ({
   );
 
   const shouldShowToggleButton: boolean =
-    displayedMainSkillsCount < totalMainSkillsCount;
+    showAll || displayedMainSkillsCount < totalMainSkillsCount;
 
   function toggleShowAll(): void {
     setShowAll(!showAll);