danvics/openreader
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix pdf loader flash

Browse source
This commit is contained in:
Richard R 2026-06-01 04:54:12 -06:00
parent 29d770941a
commit cc0d460f6b
1 changed files with 21 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

26
src/app/(app)/pdf/[id]/page.tsx
View file

@ -39,6 +39,7 @@ const PDFViewer = dynamic(
); );
const PARSE_LOADER_EXPAND_DELAY_MS = 320; const PARSE_LOADER_EXPAND_DELAY_MS = 320;
const INITIAL_LOADER_DELAY_MS = 180;
export default function PDFViewerPage() { export default function PDFViewerPage() {
const canExportAudiobook = useFeatureFlag('enableAudiobookExport'); const canExportAudiobook = useFeatureFlag('enableAudiobookExport');
@ -70,6 +71,7 @@ export default function PDFViewerPage() {
const [activeSidebar, setActiveSidebar] = useState<null | 'settings' | 'audiobook' | 'segments'>(null); const [activeSidebar, setActiveSidebar] = useState<null | 'settings' | 'audiobook' | 'segments'>(null);
const [showForceReparseConfirm, setShowForceReparseConfirm] = useState(false); const [showForceReparseConfirm, setShowForceReparseConfirm] = useState(false);
const [showDetailedParseLoader, setShowDetailedParseLoader] = useState(false); const [showDetailedParseLoader, setShowDetailedParseLoader] = useState(false);
const [showInitialLoader, setShowInitialLoader] = useState(false);
const [containerHeight, setContainerHeight] = useState<string>('auto'); const [containerHeight, setContainerHeight] = useState<string>('auto');
const inFlightDocIdRef = useRef<string | null>(null); const inFlightDocIdRef = useRef<string | null>(null);
const loadedDocIdRef = useRef<string | null>(null); const loadedDocIdRef = useRef<string | null>(null);
@ -90,6 +92,7 @@ export default function PDFViewerPage() {
setIsPdfViewerReady(false); setIsPdfViewerReady(false);
setError(null); setError(null);
setActiveSidebar(null); setActiveSidebar(null);
setShowInitialLoader(false);
inFlightDocIdRef.current = null; inFlightDocIdRef.current = null;
loadedDocIdRef.current = null; loadedDocIdRef.current = null;
}, [id]); }, [id]);
@ -155,6 +158,17 @@ export default function PDFViewerPage() {
useUnmountCleanupRef(clearCurrDoc); useUnmountCleanupRef(clearCurrDoc);
useEffect(() => {
if (!isLoading) {
setShowInitialLoader(false);
return;
}
const timeout = window.setTimeout(() => {
setShowInitialLoader(true);
}, INITIAL_LOADER_DELAY_MS);
return () => window.clearTimeout(timeout);
}, [isLoading, id]);
useEffect(() => { useEffect(() => {
if (isLoading) return; if (isLoading) return;
if (isParseReady) return; if (isParseReady) return;
@ -281,6 +295,7 @@ export default function PDFViewerPage() {
: 0; : 0;
const hasMeasuredProgress = totalPages > 0; const hasMeasuredProgress = totalPages > 0;
const isMerging = parseProgress?.phase === 'merge'; const isMerging = parseProgress?.phase === 'merge';
const compactProgressWidth = hasMeasuredProgress ? `${progressPercent}%` : '24%';
let statusText = 'Loading PDF...'; let statusText = 'Loading PDF...';
let statusSubText = 'Initializing document renderer'; let statusSubText = 'Initializing document renderer';
@ -365,10 +380,11 @@ export default function PDFViewerPage() {
<LoadingSpinner className="h-3.5 w-3.5 text-accent" /> <LoadingSpinner className="h-3.5 w-3.5 text-accent" />
</span> </span>
</div> </div>
<div className="mt-3 grid grid-cols-3 gap-1.5"> <div className="mt-3 h-1.5 overflow-hidden rounded-full bg-surface-solid">
<span className="h-1.5 rounded-full bg-accent-wash animate-pulse" /> <div
<span className="h-1.5 rounded-full bg-accent-wash animate-pulse [animation-delay:120ms]" /> className={`h-full rounded-full bg-accent transition duration-slow ease-standard ${hasMeasuredProgress ? '' : 'animate-pulse'}`}
<span className="h-1.5 rounded-full bg-accent-wash animate-pulse [animation-delay:220ms]" /> style={{ width: compactProgressWidth }}
/>
</div> </div>
</div> </div>
)} )}
@ -423,7 +439,7 @@ export default function PDFViewerPage() {
/> />
</div> </div>
) : null} ) : null}
{isLoading || !isParseReady || !isPdfViewerReady ? ( {(!isLoading || showInitialLoader) && (isLoading || !isParseReady || !isPdfViewerReady) ? (
<div className="absolute inset-0 z-10" data-testid="pdf-status-loader"> <div className="absolute inset-0 z-10" data-testid="pdf-status-loader">
{renderPdfStatusLoader()} {renderPdfStatusLoader()}
</div> </div>