From 8060766638ef43dec8456a2dd4af2aa7a11ea27d Mon Sep 17 00:00:00 2001
From: Valentin Kaelin <vkaelin@hey.com>
Date: Thu, 21 Sep 2023 14:12:11 +0200
Subject: [PATCH] fix: display/hide detailed runes

---
 client/src/components/Match/DetailedMatchTeam.vue  |  4 ++--
 .../src/components/Match/Runes/RunesContainer.vue  |  6 +++---
 client/src/components/Summoner/Live/LiveTeam.vue   |  4 ++--
 client/src/store/modules/cdragon.js                | 14 ++++++++++----
 4 files changed, 17 insertions(+), 11 deletions(-)

diff --git a/client/src/components/Match/DetailedMatchTeam.vue b/client/src/components/Match/DetailedMatchTeam.vue
index 0b9b7c5..a143818 100644
--- a/client/src/components/Match/DetailedMatchTeam.vue
+++ b/client/src/components/Match/DetailedMatchTeam.vue
@@ -395,9 +395,9 @@ export default {
         return
       }
 
-      this.displayOrHideRunes(player.perks)
+      this.displayRunes(player.perks)
     },
-    ...mapActions('cdragon', ['displayOrHideRunes']),
+    ...mapActions('cdragon', ['displayRunes']),
   },
 }
 </script>
diff --git a/client/src/components/Match/Runes/RunesContainer.vue b/client/src/components/Match/Runes/RunesContainer.vue
index 9851004..f952f04 100644
--- a/client/src/components/Match/Runes/RunesContainer.vue
+++ b/client/src/components/Match/Runes/RunesContainer.vue
@@ -83,15 +83,15 @@ export default {
 
   methods: {
     close() {
-      this.displayOrHideRunes({})
+      this.hideRunes({})
     },
     handleEscape(e) {
       if (e.key === 'Esc' || e.key === 'Escape') {
-        this.displayOrHideRunes({})
+        this.hideRunes({})
       }
     },
     createCDragonAssetUrl,
-    ...mapActions('cdragon', ['displayOrHideRunes']),
+    ...mapActions('cdragon', ['hideRunes']),
   },
 }
 </script>
diff --git a/client/src/components/Summoner/Live/LiveTeam.vue b/client/src/components/Summoner/Live/LiveTeam.vue
index fe8203c..af9cb08 100644
--- a/client/src/components/Summoner/Live/LiveTeam.vue
+++ b/client/src/components/Summoner/Live/LiveTeam.vue
@@ -282,11 +282,11 @@ export default {
     },
     selectRunes(player) {
       if (!player.perks) return
-      this.displayOrHideRunes(player.perks)
+      this.displayRunes(player.perks)
     },
     getPrimarRune,
     getSecondaryRune,
-    ...mapActions('cdragon', ['displayOrHideRunes']),
+    ...mapActions('cdragon', ['displayRunes']),
   },
 }
 </script>
diff --git a/client/src/store/modules/cdragon.js b/client/src/store/modules/cdragon.js
index 2f9fd00..6c275a6 100644
--- a/client/src/store/modules/cdragon.js
+++ b/client/src/store/modules/cdragon.js
@@ -14,18 +14,24 @@ export const state = {
 }
 
 export const mutations = {
-  DISPLAY_HIDE_RUNES(state, selectedRunes) {
-    state.runesOpen = !state.runesOpen
+  DISPLAY_RUNES(state, selectedRunes) {
+    state.runesOpen = true
     state.selectedRunes = selectedRunes
   },
+  HIDE_RUNES(state) {
+    state.runesOpen = false
+  },
   SET_RUNES(state, runes) {
     state.runes = runes
   },
 }
 
 export const actions = {
-  displayOrHideRunes({ commit }, selectedRunes) {
-    commit('DISPLAY_HIDE_RUNES', selectedRunes)
+  displayRunes({ commit }, selectedRunes) {
+    commit('DISPLAY_RUNES', selectedRunes)
+  },
+  hideRunes({ commit }) {
+    commit('HIDE_RUNES')
   },
   async getRunes({ commit, getters }) {
     if (getters.runesLoaded) {