Skip to content

Commit 9ddb05e

Browse files
committed
chore: commit local changes before switching to PR #2051
1 parent 2bedd98 commit 9ddb05e

File tree

2 files changed

+38
-28
lines changed

2 files changed

+38
-28
lines changed

internal-packages/workflow-designer-ui/src/editor/properties-panel/trigger-node-properties-panel/providers/github-trigger/components/event-selection-step.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@ export function EventSelectionStep({ onSelectEvent }: EventSelectionStepProps) {
3737
<span className="text-inverse font-medium text-[14px] truncate">
3838
{githubTrigger.event.label}
3939
</span>
40-
<span className="text-inverse/50 text-[10px] truncate group-hover:text-inverse transition-colors pr-6">
40+
<span className="text-text-muted text-[10px] truncate group-hover:text-inverse transition-colors pr-6">
4141
{`Trigger when ${githubTrigger.event.label.toLowerCase()} in your repository`}
4242
</span>
4343
</div>

internal-packages/workflow-designer-ui/src/editor/properties-panel/trigger-node-properties-panel/providers/github-trigger/github-trigger-properties-panel.tsx

Lines changed: 37 additions & 27 deletions
Original file line numberDiff line numberDiff line change
@@ -32,6 +32,7 @@ import { GitHubTriggerConfiguredView } from "../../ui";
3232
import { GitHubTriggerReconfiguringView } from "../../ui/reconfiguring-views/github-trigger-reconfiguring-view";
3333
import { EventSelectionStep } from "./components/event-selection-step";
3434
import { EventTypeDisplay } from "./components/event-type-display";
35+
import { SettingDetail, SettingLabel } from "@giselle-internal/ui/setting-label";
3536
import { InstallGitHubApplication } from "./components/install-application";
3637
import { LabelsInputStep } from "./components/labels-input-step";
3738
import { RepositoryDisplay } from "./components/repository-display";
@@ -348,39 +349,48 @@ export function Installed({
348349

349350
{step.state === "select-repository" && (
350351
<div className="overflow-y-auto flex-1 pr-2 custom-scrollbar h-full relative">
351-
<p className="text-[14px] text-[#F7F9FD] mb-2">Event type</p>
352-
<EventTypeDisplay
353-
eventId={step.eventId}
354-
className="mb-4"
355-
showDescription={false}
356-
/>
357-
358-
<p className="text-[14px] text-[#F7F9FD] mb-3">Organization</p>
359-
<div className="px-[4px] py-[4px]">
360-
<SelectRepository
361-
installations={installations}
362-
installationUrl={installationUrl}
363-
onSelectRepository={(value) => {
364-
setStep({
365-
state: "confirm-repository",
366-
eventId: step.eventId,
367-
installationId: value.installationId,
368-
owner: value.owner,
369-
repo: value.repo,
370-
repoNodeId: value.repoNodeId,
371-
});
372-
}}
373-
/>
374-
</div>
352+
<SettingLabel className="mb-[4px]">GitHub Setting</SettingLabel>
353+
<div className="flex w-full items-center justify-between gap-[12px] mb-2">
354+
<div className="shrink-0 w-[120px]"><SettingDetail className="mb-0">Event Type</SettingDetail></div>
355+
<div className="grow min-w-0 flex justify-end">
356+
<EventTypeDisplay eventId={step.eventId} className="mb-0" showDescription={false} />
357+
</div>
358+
</div>
359+
360+
361+
<div className="flex w-full items-center justify-between gap-[12px] mb-3">
362+
<div className="shrink-0 w-[120px]"><SettingDetail className="mb-0">Organization</SettingDetail></div>
363+
<div className="grow min-w-0 flex justify-end px-[4px] py-[4px]">
364+
<SelectRepository
365+
installations={installations}
366+
installationUrl={installationUrl}
367+
onSelectRepository={(value) => {
368+
setStep({
369+
state: "confirm-repository",
370+
eventId: step.eventId,
371+
installationId: value.installationId,
372+
owner: value.owner,
373+
repo: value.repo,
374+
repoNodeId: value.repoNodeId,
375+
});
376+
}}
377+
/>
378+
</div>
379+
</div>
375380
</div>
376381
)}
377382

378383
{step.state === "confirm-repository" && (
379384
<div className="overflow-y-auto flex-1 pr-2 custom-scrollbar h-full relative">
380385
<div className="flex flex-col gap-[8px]">
381-
<p className="text-[14px] text-[#F7F9FD] mb-2">Event type</p>
382-
<EventTypeDisplay eventId={step.eventId} showDescription={false} />
383-
<p className="text-[14px] text-[#F7F9FD] mb-2 mt-4">Repository</p>
386+
<SettingDetail className="mb-1">Event setting</SettingDetail>
387+
<div className="flex w-full items-center justify-between gap-[12px]">
388+
<div className="shrink-0 w-[120px]"><SettingDetail className="mb-0">Event Type</SettingDetail></div>
389+
<div className="grow min-w-0 flex justify-end">
390+
<EventTypeDisplay eventId={step.eventId} showDescription={false} />
391+
</div>
392+
</div>
393+
<SettingDetail className="mb-2 mt-4">Repository</SettingDetail>
384394
<RepositoryDisplay
385395
owner={step.owner}
386396
repo={step.repo}

0 commit comments

Comments
 (0)