From dfc699e69cc8faf4fadfe83d77e4a52829e18ad5 Mon Sep 17 00:00:00 2001 From: Toshihiko Yanase Date: Wed, 29 Nov 2023 18:25:00 +0900 Subject: [PATCH] Update code comments. --- .../ts/components/WaveSurferArtifactViewer.tsx | 9 +-------- 1 file changed, 1 insertion(+), 8 deletions(-) diff --git a/optuna_dashboard/ts/components/WaveSurferArtifactViewer.tsx b/optuna_dashboard/ts/components/WaveSurferArtifactViewer.tsx index 44dc6455..65d87bfb 100644 --- a/optuna_dashboard/ts/components/WaveSurferArtifactViewer.tsx +++ b/optuna_dashboard/ts/components/WaveSurferArtifactViewer.tsx @@ -9,15 +9,12 @@ interface WaveSurferArtifactViewerProps { url: string } -// WaveSurfer hook const useWavesurfer = ( containerRef: React.MutableRefObject, options: WaveSurferArtifactViewerProps ) => { const [wavesurfer, setWavesurfer] = useState(null) - // Initialize wavesurfer when the container mounts - // or any of the props change useEffect(() => { if (!containerRef.current) return @@ -36,8 +33,7 @@ const useWavesurfer = ( return wavesurfer } -// Create a React component that will render wavesurfer. -// Props are wavesurfer options. +// Create a React component of wavesurfer. export const WaveSurferArtifactViewer: React.FC< WaveSurferArtifactViewerProps > = (props) => { @@ -45,14 +41,11 @@ export const WaveSurferArtifactViewer: React.FC< const [isPlaying, setIsPlaying] = useState(false) const wavesurfer = useWavesurfer(containerRef, props) - // On play button click const onPlayClick = useCallback(() => { if (!wavesurfer) return wavesurfer.isPlaying() ? wavesurfer.pause() : wavesurfer.play() }, [wavesurfer]) - // Initialize wavesurfer when the container mounts - // or any of the props change useEffect(() => { if (!wavesurfer) return