AccountTimeline: improve loading spinner

environments/review-account-re-xv4rvn/deployments/753
Alex Gleason 2 years ago
parent eccfdb90f8
commit e4c829da76
No known key found for this signature in database
GPG Key ID: 7211D1F99744FBB7

@ -1,4 +1,4 @@
import React, { useEffect } from 'react'; import React, { useEffect, useState } from 'react';
import { FormattedMessage } from 'react-intl'; import { FormattedMessage } from 'react-intl';
import { useHistory } from 'react-router-dom'; import { useHistory } from 'react-router-dom';
@ -28,6 +28,7 @@ const AccountTimeline: React.FC<IAccountTimeline> = ({ params, withReplies = fal
const soapboxConfig = useSoapboxConfig(); const soapboxConfig = useSoapboxConfig();
const account = useAppSelector(state => findAccountByUsername(state, params.username)); const account = useAppSelector(state => findAccountByUsername(state, params.username));
const [accountLoading, setAccountLoading] = useState<boolean>(!account);
const path = withReplies ? `${account?.id}:with_replies` : account?.id; const path = withReplies ? `${account?.id}:with_replies` : account?.id;
const showPins = settings.getIn(['account_timeline', 'shows', 'pinned']) === true && !withReplies; const showPins = settings.getIn(['account_timeline', 'shows', 'pinned']) === true && !withReplies;
@ -43,7 +44,9 @@ const AccountTimeline: React.FC<IAccountTimeline> = ({ params, withReplies = fal
const accountUsername = account?.username || params.username; const accountUsername = account?.username || params.username;
useEffect(() => { useEffect(() => {
dispatch(fetchAccountByUsername(params.username, history)); dispatch(fetchAccountByUsername(params.username, history))
.then(() => setAccountLoading(false))
.catch(() => setAccountLoading(false));
}, [params.username]); }, [params.username]);
useEffect(() => { useEffect(() => {
@ -70,7 +73,7 @@ const AccountTimeline: React.FC<IAccountTimeline> = ({ params, withReplies = fal
} }
}; };
if (!account && isLoading) { if (!account && accountLoading) {
return <Spinner />; return <Spinner />;
} else if (!account) { } else if (!account) {
return <MissingIndicator nested />; return <MissingIndicator nested />;

Loading…
Cancel
Save