Remove debug logging, keep audio duration fix
This commit is contained in:
parent
7cef4a6aaa
commit
080b4bdaf1
1 changed files with 3 additions and 20 deletions
|
|
@ -4,7 +4,6 @@ const { useState: useStateRD, useEffect: useEffectRD, useRef: useRefRD, useCallb
|
|||
const API_BASE = '';
|
||||
|
||||
function RecordingDetail({ recording: r, data, onBack, allHighlights, setAllHighlights, initialSeekTime, onSeekHandled }) {
|
||||
console.log('[RecordingDetail] render, recording id:', r?.id, 'r.durationMinutes:', r?.durationMinutes);
|
||||
const [currentTime, setCurrentTime] = useStateRD(0); // seconds into recording
|
||||
const [playing, setPlaying] = useStateRD(false);
|
||||
const [speed, setSpeed] = useStateRD(1.0);
|
||||
|
|
@ -19,7 +18,6 @@ function RecordingDetail({ recording: r, data, onBack, allHighlights, setAllHigh
|
|||
// Use actual audio duration if available, fallback to metadata
|
||||
const metadataDuration = r.durationMinutes * 60;
|
||||
const totalSeconds = audioDuration || metadataDuration || 1; // avoid division by zero
|
||||
console.log('[duration] audioDuration:', audioDuration, 'r.durationMinutes:', r.durationMinutes, 'totalSeconds:', totalSeconds);
|
||||
const recHighlights = allHighlights.filter(h => h.recordingId === r.id);
|
||||
|
||||
// Audio URL from API
|
||||
|
|
@ -82,27 +80,16 @@ function RecordingDetail({ recording: r, data, onBack, allHighlights, setAllHigh
|
|||
const audio = audioRef.current;
|
||||
if (!audio) return;
|
||||
|
||||
const handleTimeUpdate = () => {
|
||||
console.log('[audio] timeupdate:', audio.currentTime, 'duration:', audio.duration);
|
||||
setCurrentTime(audio.currentTime);
|
||||
};
|
||||
const handleTimeUpdate = () => setCurrentTime(audio.currentTime);
|
||||
const handleEnded = () => setPlaying(false);
|
||||
const handlePlay = () => {
|
||||
console.log('[audio] play event');
|
||||
setPlaying(true);
|
||||
};
|
||||
const handlePause = () => {
|
||||
console.log('[audio] pause event');
|
||||
setPlaying(false);
|
||||
};
|
||||
const handlePlay = () => setPlaying(true);
|
||||
const handlePause = () => setPlaying(false);
|
||||
const handleDurationChange = () => {
|
||||
console.log('[audio] durationchange:', audio.duration, 'isFinite:', isFinite(audio.duration));
|
||||
if (audio.duration && isFinite(audio.duration)) {
|
||||
setAudioDuration(audio.duration);
|
||||
}
|
||||
};
|
||||
const handleLoadedMetadata = () => {
|
||||
console.log('[audio] loadedmetadata - duration:', audio.duration, 'readyState:', audio.readyState);
|
||||
if (audio.duration && isFinite(audio.duration)) {
|
||||
setAudioDuration(audio.duration);
|
||||
}
|
||||
|
|
@ -116,7 +103,6 @@ function RecordingDetail({ recording: r, data, onBack, allHighlights, setAllHigh
|
|||
audio.addEventListener('durationchange', handleDurationChange);
|
||||
|
||||
// Check if duration is already available
|
||||
console.log('[audio] initial check - duration:', audio.duration, 'readyState:', audio.readyState);
|
||||
if (audio.duration && isFinite(audio.duration)) {
|
||||
setAudioDuration(audio.duration);
|
||||
}
|
||||
|
|
@ -157,10 +143,8 @@ function RecordingDetail({ recording: r, data, onBack, allHighlights, setAllHigh
|
|||
const rect = e.currentTarget.getBoundingClientRect();
|
||||
const pct = (e.clientX - rect.left) / rect.width;
|
||||
const newTime = pct * totalSeconds;
|
||||
console.log('[seek] clicked pct:', pct, 'newTime:', newTime, 'totalSeconds:', totalSeconds, 'audioDuration:', audioDuration);
|
||||
if (audioRef.current) {
|
||||
audioRef.current.currentTime = newTime;
|
||||
console.log('[seek] set audio.currentTime to:', audioRef.current.currentTime);
|
||||
}
|
||||
setCurrentTime(newTime);
|
||||
}
|
||||
|
|
@ -266,7 +250,6 @@ function RecordingDetail({ recording: r, data, onBack, allHighlights, setAllHigh
|
|||
};
|
||||
|
||||
const progressPct = (currentTime / totalSeconds) * 100;
|
||||
console.log('[render] currentTime:', currentTime, 'totalSeconds:', totalSeconds, 'progressPct:', progressPct, 'audioDuration state:', audioDuration);
|
||||
|
||||
// Waveform: fake bars
|
||||
const waveformBars = 120;
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue