fix: several UI fixes for Template Builder (#26903)

- Module selection summary now has `rounded-sm` corners to fit the icon
- Modules without configuration now say so with a check mark
- Correct "Additional settings" label
This commit is contained in:
Jeremy Ruppel
2026-06-30 20:01:48 -04:00
committed by GitHub
parent c56a8f1d99
commit a73e677639
2 changed files with 9 additions and 4 deletions
@@ -1,4 +1,4 @@
import { TrashIcon } from "lucide-react"; import { CheckIcon, TrashIcon } from "lucide-react";
import { Button } from "#/components/Button/Button"; import { Button } from "#/components/Button/Button";
import { CollapsibleSummary } from "#/components/CollapsibleSummary/CollapsibleSummary"; import { CollapsibleSummary } from "#/components/CollapsibleSummary/CollapsibleSummary";
import { TemplateBuilderAvatarData } from "#/pages/TemplateBuilder/TemplateBuilderAvatarData"; import { TemplateBuilderAvatarData } from "#/pages/TemplateBuilder/TemplateBuilderAvatarData";
@@ -58,14 +58,19 @@ export const ModuleConfiguration: React.FC<ModuleConfigurationProps> = ({
</ConfigurationFieldContainer> </ConfigurationFieldContainer>
)} )}
{optionalFields && optionalFields.length > 0 && ( {optionalFields && optionalFields.length > 0 ? (
<CollapsibleSummary label="Advanced settings" className="mt-4"> <CollapsibleSummary label="Additional settings" className="mt-4">
<ConfigurationFieldContainer> <ConfigurationFieldContainer>
{optionalFields.map((f) => ( {optionalFields.map((f) => (
<ConfigurationField key={f.id} field={f} /> <ConfigurationField key={f.id} field={f} />
))} ))}
</ConfigurationFieldContainer> </ConfigurationFieldContainer>
</CollapsibleSummary> </CollapsibleSummary>
) : (
<div className="text-sm text-content-secondary flex items-center gap-2 mt-4">
<CheckIcon className="w-4 h-4" />
No configuration required.
</div>
)} )}
</section> </section>
); );
@@ -174,7 +174,7 @@ const ModuleSelection: React.FC<ModuleSelectionProps> = ({
{modules.map((module) => ( {modules.map((module) => (
<div <div
key={module.id} key={module.id}
className="group flex items-start justify-between p-1 mb-1 hover:bg-surface-secondary" className="group flex items-start justify-between p-1 mb-1 rounded-sm hover:bg-surface-secondary"
> >
<div className="h-[1lh] content-center"> <div className="h-[1lh] content-center">
<img <img