jyapayne/langflow
1
0
Fork 0
Code Issues Pull requests Projects Releases 1 Packages Wiki Activity Actions

refactor(extraSidebarComponent): extract SidebarDraggableComponent to a separate file for reusability and better organization

Browse source 
The `ExtraSidebar` component in `extraSidebarComponent/index.tsx` has been refactored to extract the `SidebarDraggableComponent` into a separate file. This was done to improve code organization and reusability. The extracted component takes props such as `display_name`, `itemName`, `error`, `color`, and `onDragStart` to render the draggable sidebar component.
This commit is contained in:
anovazzi1 2023-10-02 18:19:43 -03:00
commit d950b6e252
2 changed files with 60 additions and 39 deletions
Download patch file Download diff file Expand all files Collapse all files

57
src/frontend/src/pages/FlowPage/components/extraSidebarComponent/index.tsx
View file

@ -17,6 +17,7 @@ import {
} from "../../../../utils/styleUtils";
import { classNames, removeCountFromString } from "../../../../utils/utils";
import DisclosureComponent from "../DisclosureComponent";
import SidebarDraggableComponent from "./sideBarDraggableComponent";
export default function ExtraSidebar(): JSX.Element {
const { data, templates, getFilterEdge, setFilterEdge } =
@ -242,45 +243,23 @@ export default function ExtraSidebar(): JSX.Element {
side="right"
key={index}
>
<div key={SBItemName} data-tooltip-id={SBItemName}>
<div
draggable={!data[SBSectionName][SBItemName].error}
className={
"side-bar-components-border bg-background" +
(data[SBSectionName][SBItemName].error
? " cursor-not-allowed select-none"
: "")
}
style={{
borderLeftColor:
nodeColors[SBSectionName] ?? nodeColors.unknown,
}}
onDragStart={(event) =>
onDragStart(event, {
//split type to remove type in nodes saved with same name removing it's
type: removeCountFromString(SBItemName),
node: data[SBSectionName][SBItemName],
})
}
onDragEnd={() => {
document.body.removeChild(
document.getElementsByClassName(
"cursor-grabbing"
)[0]
);
}}
>
<div className="side-bar-components-div-form">
<span className="side-bar-components-text">
{data[SBSectionName][SBItemName].display_name}
</span>
<IconComponent
name="Menu"
className="side-bar-components-icon "
/>
</div>
</div>
</div>
<SidebarDraggableComponent
key={SBItemName}
onDragStart={(event) =>
onDragStart(event, {
//split type to remove type in nodes saved with same name removing it's
type: removeCountFromString(SBItemName),
node: data[SBSectionName][SBItemName],
})
}
color={nodeColors[SBSectionName]}
itemName={SBItemName}
//convert error to boolean
error={!!data[SBSectionName][SBItemName].error}
display_name={
data[SBSectionName][SBItemName].display_name
}
/>
</ShadTooltip>
))}
</div>

42
src/frontend/src/pages/FlowPage/components/extraSidebarComponent/sideBarDraggableComponent/index.tsx Normal file
View file

@ -0,0 +1,42 @@
import { DragEventHandler } from "react";
import IconComponent from "../../../../../components/genericIconComponent";
export default function SidebarDraggableComponent({
display_name,
itemName,
error,
color,
onDragStart,
}: {
display_name: string;
itemName: string;
error: boolean;
color: string;
onDragStart: DragEventHandler<HTMLDivElement>;
}) {
return (
<div key={itemName} data-tooltip-id={itemName}>
<div
draggable={!error}
className={
"side-bar-components-border bg-background" +
(error ? " cursor-not-allowed select-none" : "")
}
style={{
borderLeftColor: color,
}}
onDragStart={onDragStart}
onDragEnd={() => {
document.body.removeChild(
document.getElementsByClassName("cursor-grabbing")[0]
);
}}
>
<div className="side-bar-components-div-form">
<span className="side-bar-components-text">{display_name}</span>
<IconComponent name="Menu" className="side-bar-components-icon " />
</div>
</div>
</div>
);
}