🐛 fix(GenericNode/index.tsx): rename getIconComponent to getIconPlayOrPauseComponent for better clarity and semantics

🐛 fix(GenericNode/index.tsx): rename renderIconComponents to renderIconPlayOrPauseComponents for better clarity and semantics
🐛 fix(GenericNode/index.tsx): rename getSpecificClass to getSpecificClassFromBuildStatus for better clarity and semantics
🐛 fix(GenericNode/index.tsx): rename getNodeClassName to getNodeBorderClassName for better clarity and semantics
This commit is contained in:
cristhianzl 2024-02-20 10:48:32 -03:00
commit b3f82872b0

View file

@ -157,7 +157,7 @@ export default function GenericNode({
); );
}; };
const getIconComponent = (name, className) => ( const getIconPlayOrPauseComponent = (name, className) => (
<IconComponent <IconComponent
name={name} name={name}
className={`absolute h-5 stroke-2 ${className} ml-0.5`} className={`absolute h-5 stroke-2 ${className} ml-0.5`}
@ -179,20 +179,20 @@ export default function GenericNode({
} }
}; };
const renderIconComponents = ( const renderIconPlayOrPauseComponents = (
buildStatus: BuildStatus | undefined, buildStatus: BuildStatus | undefined,
validationStatus: validationStatusType | null, validationStatus: validationStatusType | null,
isBuilding: boolean isBuilding: boolean
) => { ) => {
if (buildStatus === BuildStatus.BUILDING) { if (buildStatus === BuildStatus.BUILDING) {
return getIconComponent("Square", "red-status"); return getIconPlayOrPauseComponent("Square", "red-status");
} else { } else {
const className = getStatusClassName(validationStatus, isBuilding); const className = getStatusClassName(validationStatus, isBuilding);
return <>{getIconComponent("Play", className)}</>; return <>{getIconPlayOrPauseComponent("Play", className)}</>;
} }
}; };
const getSpecificClass = ( const getSpecificClassFromBuildStatus = (
buildStatus: BuildStatus | undefined, buildStatus: BuildStatus | undefined,
validationStatus: validationStatusType | null validationStatus: validationStatusType | null
) => { ) => {
@ -209,7 +209,7 @@ export default function GenericNode({
} }
}; };
const getNodeClassName = ( const getNodeBorderClassName = (
selected: boolean, selected: boolean,
showNode: boolean, showNode: boolean,
buildStatus: BuildStatus | undefined, buildStatus: BuildStatus | undefined,
@ -219,7 +219,7 @@ export default function GenericNode({
getBaseBorderClass(selected), getBaseBorderClass(selected),
getNodeSizeClass(showNode), getNodeSizeClass(showNode),
"generic-node-div", "generic-node-div",
getSpecificClass(buildStatus, validationStatus) getSpecificClassFromBuildStatus(buildStatus, validationStatus)
); );
}; };
@ -253,7 +253,7 @@ export default function GenericNode({
</NodeToolbar> </NodeToolbar>
<div <div
className={getNodeClassName( className={getNodeBorderClassName(
selected, selected,
showNode, showNode,
data?.build_status, data?.build_status,
@ -495,7 +495,7 @@ export default function GenericNode({
} }
> >
<div className="generic-node-status-position flex items-center justify-center"> <div className="generic-node-status-position flex items-center justify-center">
{renderIconComponents( {renderIconPlayOrPauseComponents(
data?.build_status, data?.build_status,
validationStatus, validationStatus,
isBuilding isBuilding