chore: improve visuals of dynamic parameters (#18537)

<img width="343" alt="Screenshot 2025-06-24 at 16 43 22"
src="https://github.com/user-attachments/assets/dc554f63-b825-4e59-896a-d078bae51c4a"
/>
<img width="837" alt="Screenshot 2025-06-24 at 16 43 37"
src="https://github.com/user-attachments/assets/e9e88164-0c87-43b5-9eb1-63fd6ea85095"
/>
This commit is contained in:
Jaayden Halko
2025-06-24 12:57:41 -04:00
committed by GitHub
parent e5eb2a8322
commit ccf294eaf8
3 changed files with 8 additions and 9 deletions
@@ -139,7 +139,7 @@ const ParameterLabel: FC<ParameterLabelProps> = ({
htmlFor={id}
className="flex gap-2 flex-wrap text-sm font-medium"
>
<span className="flex">
<span className="flex font-semibold">
{displayName}
{parameter.required && (
<span className="text-content-destructive">*</span>
@@ -175,7 +175,8 @@ const ParameterLabel: FC<ParameterLabelProps> = ({
</span>
</TooltipTrigger>
<TooltipContent className="max-w-xs">
This parameter only applies for a single workspace start
This parameter is ephemeral and will reset to the template
default on workspace restart.
</TooltipContent>
</Tooltip>
</TooltipProvider>
@@ -209,7 +210,7 @@ const ParameterLabel: FC<ParameterLabelProps> = ({
</span>
</TooltipTrigger>
<TooltipContent className="max-w-xs">
Autofilled from the URL
Autofilled from the URL.
</TooltipContent>
</Tooltip>
</TooltipProvider>
@@ -114,12 +114,10 @@ const BuildParametersPopoverContent: FC<BuildParametersPopoverContentProps> = ({
return (
<div className="flex flex-col gap-4 p-5">
<h1 className="text-xl m-0 text-content-primary font-semibold leading-none ">
Ephemeral Parameters
</h1>
<p className="m-0 text-sm text-content-secondary">
This template has ephemeral parameters that must be configured on the
workspace parameters page
This workspace has ephemeral parameters which may use a temporary
value on workspace start. Configure the following parameters in
workspace settings.
</p>
<div>
@@ -194,7 +194,7 @@ export const WorkspaceParametersPageViewExperimental: FC<
{(templateVersionId || workspace.latest_build.template_version_id) && (
<div className="flex flex-col gap-2">
<Label className="text-sm text-content-secondary">Version ID</Label>
<p className="m-0 text-sm font-medium">
<p className="m-0 text-xs font-medium font-mono">
{templateVersionId ?? workspace.latest_build.template_version_id}
</p>
</div>