Skip to content

Commit 4245255

Browse files
authored
Removes icons next to title in run and editor (#1478)
## Description Removed unnecessary icon imports and their usage from the PipelineDetails and RunDetails components. Specifically: - Removed the Network icon and its rendering from PipelineDetails - Removed the Videotape icon and its rendering from RunDetails Do we actually need these? I personally think no icon is cleaner. Also this section has a lot of information already, so adding in this visual flare seems a little too much. ## Related Issue and Pull requests ## Type of Change - [x] Improvement - [x] Cleanup/Refactor ## Checklist - [x] I have tested this does not break current pipelines / runs functionality - [x] I have tested the changes on staging ## Screenshots (if applicable) Before ![Screenshot 2025-12-05 at 1.48.21 PM.png](https://app.graphite.com/user-attachments/assets/6e7e357b-f830-4eeb-8b2a-b58781bd52d3.png) After: ![Screenshot 2025-12-05 at 1.48.38 PM.png](https://app.graphite.com/user-attachments/assets/30ba6b73-3980-42de-92c3-e23d97c9eba2.png) ## Test Instructions Verify that the Pipeline Details and Run Details pages display correctly without the icons that were previously shown next to the pipeline/run names. ## Additional Comments This change simplifies the UI by removing decorative icons that weren't adding significant value to the interface.
1 parent 76f0c28 commit 4245255

File tree

2 files changed

+2
-4
lines changed

2 files changed

+2
-4
lines changed

src/components/Editor/PipelineDetails.tsx

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
import { Frown, Network } from "lucide-react";
1+
import { Frown } from "lucide-react";
22
import { useEffect, useState } from "react";
33

44
import { PipelineValidationList } from "@/components/Editor/components/PipelineValidationList/PipelineValidationList";
@@ -134,7 +134,6 @@ const PipelineDetails = () => {
134134
>
135135
{/* Header */}
136136
<div className="flex items-center gap-2 max-w-[90%]">
137-
<Network className="w-6 h-6 text-secondary-foreground rotate-270 min-w-6 min-h-6" />
138137
<CopyText className="text-lg font-semibold" alwaysShowButton>
139138
{componentSpec.name ?? "Unnamed Pipeline"}
140139
</CopyText>

src/components/PipelineRun/RunDetails.tsx

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
import { Frown, Videotape } from "lucide-react";
1+
import { Frown } from "lucide-react";
22

33
import { CopyText } from "@/components/shared/CopyText/CopyText";
44
import { Spinner } from "@/components/ui/spinner";
@@ -84,7 +84,6 @@ export const RunDetails = () => {
8484
return (
8585
<div className="p-2 flex flex-col gap-6 h-full">
8686
<div className="flex items-center gap-2 max-w-[90%]">
87-
<Videotape className="w-6 h-6 text-gray-500" />
8887
<CopyText className="text-lg font-semibold" alwaysShowButton>
8988
{componentSpec.name ?? "Unnamed Pipeline"}
9089
</CopyText>

0 commit comments

Comments
 (0)