@moises.ai/design-system 4.18.8 → 4.18.10

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@moises.ai/design-system",
-  "version": "4.18.8",
+  "version": "4.18.10",
   "description": "Design System package based on @radix-ui/themes with custom defaults",
   "private": false,
   "type": "module",

package/src/components/ProjectsList/ProjectTrackVoice/ProjectTrackVoice.jsx

@@ -1,5 +1,5 @@
 import { Flex, Text, Spinner } from '@radix-ui/themes'
-import { useCallback, useContext, useEffect, useMemo, useRef, useState } from 'react'
+import { useCallback, useContext, useEffect, useMemo, useState } from 'react'
 import { ArrowLeftIcon } from '../../../icons'
 import { ProjectsListContext } from '../context'
 import { NumberPicker } from '../../NumberPicker/NumberPicker'
@@ -7,11 +7,7 @@ import { UserIcon } from '../../../icons'
 import { Button } from '../../Button/Button'
 import styles from './ProjectTrackVoice.module.css'
 
-const DEBOUNCE_MS = 3000
-
-const ProjectTrackVoice = ({
-  ...props
-}) => {
+const ProjectTrackVoice = ({ ...props }) => {
   const { onClickTrack, onChangeTransfer, projects, labels = {} } = useContext(ProjectsListContext)
   const { projectId, id, title, cover, transfers, audioUrl, loading: _loading } = props
 
@@ -19,32 +15,29 @@ const ProjectTrackVoice = ({
 
   const max = 24
   const min = -24
-  const [pitchShiftSelected, setpitchShiftSelected] = useState(hasTransfers ? parseInt(transfers?.[0]?.pitchShift ?? 0) : 0)
-
-  const [loading, setLoading] = useState(_loading)
-  const debounceRef = useRef(null)
-
-  useEffect(() => {
-    if (hasTransfers && !transfers.some((transfer) => parseInt(transfer.pitchShift) === pitchShiftSelected)) {
-      setLoading(true)
-    } else {
-      setLoading(false)
-    }
-  }, [pitchShiftSelected, transfers, hasTransfers])
+  const [pitchShiftSelected, setpitchShiftSelected] = useState(null)
 
   useEffect(() => {
-    setLoading(_loading)
-
-    if (!_loading && hasTransfers) {
+    if (hasTransfers && pitchShiftSelected === null) {
       setpitchShiftSelected(parseInt(transfers?.[0]?.pitchShift ?? 0))
     }
-  }, [_loading, hasTransfers, transfers])
+  }, [hasTransfers, pitchShiftSelected])
+
+  const loading = useMemo(() => {
+    if (
+      (!hasTransfers && _loading) ||
+      (hasTransfers && pitchShiftSelected === null)
+    ) {
+      return true
+    }
 
-  useEffect(() => {
-    return () => {
-      if (debounceRef.current) clearTimeout(debounceRef.current)
+    if (pitchShiftSelected) {
+      const currentTransfer = transfers.find((transfer) => parseInt(transfer.pitchShift) === pitchShiftSelected)
+      return currentTransfer?.loading === true || !currentTransfer?.audioUrl
     }
-  }, [])
+
+    return false
+  }, [_loading, hasTransfers, pitchShiftSelected, transfers])
 
   const getAudioUrl = useCallback((pitchShift) => {
     return hasTransfers ? transfers?.find((transfer) => parseInt(transfer.pitchShift) === pitchShift)?.audioUrl : audioUrl
@@ -52,20 +45,16 @@ const ProjectTrackVoice = ({
 
   const handleChangeTransfer = useCallback((value) => {
     setpitchShiftSelected(value)
-
-    if (debounceRef.current) clearTimeout(debounceRef.current)
-    debounceRef.current = setTimeout(() => {
-      const newAudioUrl = getAudioUrl(value)
-      const project = projects.find((project) => project.id === projectId)
-      onChangeTransfer?.(project, {
-        id,
-        title,
-        audioUrl: newAudioUrl,
-        processTrack: !newAudioUrl,
-        pitchShift: value,
-      })
-    }, DEBOUNCE_MS)
-  }, [onChangeTransfer, projectId, id, title, getAudioUrl, projects])
+    const newAudioUrl = getAudioUrl(value)
+    const project = projects.find((project) => project.id === projectId)
+    onChangeTransfer?.(project, {
+      id,
+      title,
+      audioUrl: newAudioUrl,
+      processTrack: !newAudioUrl,
+      pitchShift: value,
+    })
+  }, [onChangeTransfer, projectId, id, title, transfers])
 
   const handleClickTrack = useCallback((e) => {
     e.stopPropagation()
@@ -74,18 +63,12 @@ const ProjectTrackVoice = ({
       return
     }
 
-    const processTrack = (!hasTransfers && !audioUrl)
-
-    if (processTrack) {
-      setLoading(true)
-    }
-
     onClickTrack?.({
       ...props,
       id,
       title,
       audioUrl: hasTransfers || audioUrl ? getAudioUrl(pitchShiftSelected) : null,
-      processTrack,
+      processTrack: !hasTransfers && !audioUrl,
     })
   }, [onClickTrack, props, id, title, getAudioUrl, pitchShiftSelected, hasTransfers, loading, audioUrl])
 
@@ -128,19 +111,19 @@ const ProjectTrackVoice = ({
         )}
 
         {!hasTransfers && !audioUrl && !loading ? (
-          <Flex align="center" justify="center" p="2">
+          <Flex align="center" justify="center" pr="2">
             <Button variant="soft" color="gray" size="1">
               {labels.process || 'Process'}
             </Button>
           </Flex>
-        ) : (
+        ) : hasTransfers ? (
           <NumberPicker
             min={min}
             max={max}
             value={pitchShiftSelected}
             onChange={handleChangeTransfer}
           />
-        )}
+        ) : null}
       </Flex>
     </Flex>
   )