Skip to content

Commit 718350d

Browse files
committed
chore: cleanup html
1 parent 6d69706 commit 718350d

File tree

2 files changed

+28
-24
lines changed

2 files changed

+28
-24
lines changed

site/src/components/EphemeralParametersDialog/EphemeralParametersDialog.tsx

Lines changed: 13 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -52,18 +52,20 @@ export const EphemeralParametersDialog: FC<EphemeralParametersDialogProps> = ({
5252
ephemeral parameters that will be reset to their default values
5353
</DialogDescription>
5454
<DialogDescription>
55-
{ephemeralParameters.map((param) => (
56-
<div key={param.name}>
57-
<p className="text-content-primary m-0 font-bold">
58-
{param.display_name || param.name}
59-
</p>
60-
{param.description && (
61-
<p className="m-0 text-sm text-content-secondary">
62-
{param.description}
55+
<ul className="list-none pl-6 space-y-2">
56+
{ephemeralParameters.map((param) => (
57+
<li key={param.name}>
58+
<p className="text-content-primary m-0 font-bold">
59+
{param.display_name || param.name}
6360
</p>
64-
)}
65-
</div>
66-
))}
61+
{param.description && (
62+
<p className="m-0 text-sm text-content-secondary">
63+
{param.description}
64+
</p>
65+
)}
66+
</li>
67+
))}
68+
</ul>
6769
</DialogDescription>
6870
<DialogDescription>
6971
Would you like to go to the workspace parameters page to review and

site/src/pages/WorkspacePage/WorkspaceActions/BuildParametersPopover.tsx

Lines changed: 15 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -122,19 +122,21 @@ const BuildParametersPopoverContent: FC<BuildParametersPopoverContentProps> = ({
122122
workspace parameters page
123123
</p>
124124

125-
<div className="flex flex-col gap-2">
126-
{ephemeralParameters.map((param) => (
127-
<div key={param.name} className="flex flex-col gap-2">
128-
<p className="text-content-primary m-0 font-bold">
129-
{param.display_name || param.name}
130-
</p>
131-
{param.description && (
132-
<div className="m-0 text-sm text-content-secondary">
133-
{param.description}
134-
</div>
135-
)}
136-
</div>
137-
))}
125+
<div>
126+
<ul className="list-none pl-3 space-y-2">
127+
{ephemeralParameters.map((param) => (
128+
<li key={param.name}>
129+
<p className="text-content-primary m-0 font-bold">
130+
{param.display_name || param.name}
131+
</p>
132+
{param.description && (
133+
<p className="m-0 text-sm text-content-secondary">
134+
{param.description}
135+
</p>
136+
)}
137+
</li>
138+
))}
139+
</ul>
138140
</div>
139141

140142
<Button className="w-full" onClick={handleGoToParameters}>

0 commit comments

Comments
 (0)