moon
/
soapbox
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

refactor: add TODO and FIXME comments

This commit is contained in:
P. Reis 2024-12-09 14:14:47 -03:00
parent b03c29c371
commit 9bd964880b
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
src/components/pure-status.tsx
View File

@ -127,7 +127,7 @@ const PureStatus: React.FC<IPureStatus> = (props) => {
} }
}, [overlay.current]); }, [overlay.current]);
// START: this is a temporary code, it will be removed // TODO: remove this code, it will be removed once all components in this file are pure.
useEffect(() => { useEffect(() => {
dispatch(importFetchedStatuses([status])); dispatch(importFetchedStatuses([status]));
}, []); }, []);
@ -136,7 +136,7 @@ const PureStatus: React.FC<IPureStatus> = (props) => {
if (!statusImmutable) { if (!statusImmutable) {
return null; return null;
} }
// END: this is a temporary code, it will be removed // END TODO
const handleToggleMediaVisibility = (): void => { const handleToggleMediaVisibility = (): void => {
setShowMedia(!showMedia); setShowMedia(!showMedia);
@ -486,7 +486,7 @@ const PureStatus: React.FC<IPureStatus> = (props) => {
{(quote || actualStatus.card || actualStatus.media_attachments.length > 0) && ( {(quote || actualStatus.card || actualStatus.media_attachments.length > 0) && (
<Stack space={4}> <Stack space={4}>
<StatusMedia <StatusMedia
status={statusImmutable} // fix later status={statusImmutable} // FIXME: stop using 'statusImmutable' and use 'status' variable directly, for that create a new component called 'PureStatusMedia'
muted={muted} muted={muted}
onClick={handleClick} onClick={handleClick}
showMedia={showMedia} showMedia={showMedia}
@ -502,7 +502,7 @@ const PureStatus: React.FC<IPureStatus> = (props) => {
{(!hideActionBar && !isUnderReview) && ( {(!hideActionBar && !isUnderReview) && (
<div className='pt-4'> <div className='pt-4'>
<StatusActionBar status={statusImmutable} /> {/* fix later */} <StatusActionBar status={statusImmutable} /> {/* FIXME: stop using 'statusImmutable' and use 'status' variable directly, for that create a new component called 'PureStatusActionBar' */}
</div> </div>
)} )}
</div> </div>