Merged PR 4366: Fixed setVolume call

Fixed setVolume call
Kryha · Jun 14, 2023 · 06239b3 · 06239b3
1 parent a1ca295
commit 06239b3
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/frontend/src/components/top-navigation/top-navigation.tsx b/frontend/src/components/top-navigation/top-navigation.tsx
@@ -21,10 +21,11 @@ export type ActiveDropdown = "rules" | "menu" | "sound" | undefined;
 export const TopNavigation: FC<Props> = ({ location }) => {
   const [activeDropdown, setActiveDropdown] = useState<ActiveDropdown>();
   const [isComponentVisible, setIsComponentVisible] = useState(false);
-  const masterVolume = useStore((state) => state.masterVolume);
-  const setVolume = useChangeVolume();
-  const ref = useRef(null);
+
+  const { setVolume, masterVolume } = useChangeVolume();
+
   const isMobile = useIsMobile();
+  const ref = useRef(null);
 
   const handleDropdownClick = (dropdown: ActiveDropdown) => {
     if (activeDropdown === dropdown) {

diff --git a/frontend/src/hooks/use-sound.ts b/frontend/src/hooks/use-sound.ts
@@ -5,7 +5,8 @@ const audioContext = new AudioContext();
 const masterGainNode = audioContext.createGain();
 
 export const useChangeVolume = () => {
-  const { masterVolume, setMasterVolume } = useStore();
+  const masterVolume = useStore((state) => state.masterVolume);
+  const setMasterVolume = useStore((state) => state.setMasterVolume);
 
   useEffect(() => {
     masterGainNode.gain.setValueAtTime(masterVolume, 0);