Skip to content

Commit 85d3acd

Browse files
BillLeoutsakosvl346Bill Leoutsakos
BillLeoutsakosvl346
and
Bill Leoutsakos
authored
improvement(ui): standardize card header icon actions (#8033)
Co-authored-by: Bill Leoutsakos <billleoutsakos@Bills-MacBook-Pro.local>
1 parent 075e236 commit 85d3acd

10 files changed

Lines changed: 22 additions & 30 deletions

File tree

‎apps/sim/app/workspace/[workspaceId]/w/[workflowId]/components/panel/components/editor/components/sub-block/components/condition-input/condition-input.tsx‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -969,7 +969,7 @@ export function ConditionInput({
969969
disabled ||
970970
(!isRouterMode && isElseConditionTitle(block.title))
971971
}
972-
className='h-auto p-0'
972+
size='icon'
973973
>
974974
<Plus className='size-[14px]' />
975975
<span className='sr-only'>Add Block</span>
@@ -989,7 +989,7 @@ export function ConditionInput({
989989
disabled ||
990990
(!isRouterMode && isElseConditionTitle(block.title))
991991
}
992-
className='h-auto p-0'
992+
size='icon'
993993
>
994994
<ChevronUp className='size-[14px]' />
995995
<span className='sr-only'>Move Up</span>
@@ -1011,7 +1011,7 @@ export function ConditionInput({
10111011
isElseConditionTitle(conditionalBlocks[index + 1]?.title)) ||
10121012
(!isRouterMode && isElseConditionTitle(block.title))
10131013
}
1014-
className='h-auto p-0'
1014+
size='icon'
10151015
>
10161016
<ChevronDown className='size-[14px]' />
10171017
<span className='sr-only'>Move Down</span>
@@ -1028,7 +1028,7 @@ export function ConditionInput({
10281028
disabled={
10291029
isPreview || disabled || conditionalBlocks.length <= (isRouterMode ? 1 : 2)
10301030
}
1031-
className='h-auto p-0'
1031+
size='icon'
10321032
>
10331033
<Trash className='size-[14px]' />
10341034
<span className='sr-only'>Delete Block</span>

‎apps/sim/app/workspace/[workspaceId]/w/[workflowId]/components/panel/components/editor/components/sub-block/components/document-tag-entry/document-tag-entry.tsx‎

Lines changed: 2 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -257,20 +257,15 @@ export function DocumentTagEntry({
257257

258258
const renderActions = (tag: DocumentTag) => (
259259
<>
260-
<Button
261-
variant='ghost'
262-
onClick={addTag}
263-
disabled={isReadOnly || !canAddMoreTags}
264-
className='h-auto p-0'
265-
>
260+
<Button variant='ghost' onClick={addTag} disabled={isReadOnly || !canAddMoreTags} size='icon'>
266261
<Plus className='size-[14px]' />
267262
<span className='sr-only'>Add Tag</span>
268263
</Button>
269264
<Button
270265
variant='ghost-destructive'
271266
onClick={() => removeTag(tag.id)}
272267
disabled={isReadOnly}
273-
className='h-auto p-0'
268+
size='icon'
274269
>
275270
<Trash className='size-[14px]' />
276271
<span className='sr-only'>Delete Tag</span>

‎apps/sim/app/workspace/[workspaceId]/w/[workflowId]/components/panel/components/editor/components/sub-block/components/eval-input/eval-input.tsx‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -147,7 +147,7 @@ export function EvalInput({
147147
variant='ghost'
148148
onClick={addMetric}
149149
disabled={isPreview || disabled}
150-
className='h-auto p-0'
150+
size='icon'
151151
>
152152
<Plus className='size-[14px]' />
153153
<span className='sr-only'>Add Metric</span>
@@ -162,7 +162,7 @@ export function EvalInput({
162162
variant='ghost-destructive'
163163
onClick={() => removeMetric(metric.id)}
164164
disabled={isPreview || disabled || metrics.length === 1}
165-
className='h-auto p-0'
165+
size='icon'
166166
>
167167
<Trash className='size-[14px]' />
168168
<span className='sr-only'>Delete Metric</span>

‎apps/sim/app/workspace/[workspaceId]/w/[workflowId]/components/panel/components/editor/components/sub-block/components/filter-builder/components/filter-rule-row.tsx‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -129,15 +129,15 @@ export function FilterRuleRow({
129129

130130
const renderActions = () => (
131131
<>
132-
<Button variant='ghost' onClick={onAdd} disabled={isReadOnly} className='h-auto p-0'>
132+
<Button variant='ghost' onClick={onAdd} disabled={isReadOnly} size='icon'>
133133
<Plus className='size-[14px]' />
134134
<span className='sr-only'>Add Condition</span>
135135
</Button>
136136
<Button
137137
variant='ghost-destructive'
138138
onClick={() => onRemove(rule.id)}
139139
disabled={isReadOnly}
140-
className='h-auto p-0'
140+
size='icon'
141141
>
142142
<Trash className='size-[14px]' />
143143
<span className='sr-only'>Delete Condition</span>

‎apps/sim/app/workspace/[workspaceId]/w/[workflowId]/components/panel/components/editor/components/sub-block/components/knowledge-tag-filters/knowledge-tag-filters.tsx‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -257,7 +257,7 @@ export function KnowledgeTagFilters({
257257
addFilter()
258258
}}
259259
disabled={isReadOnly}
260-
className='h-auto p-0'
260+
size='icon'
261261
>
262262
<Plus className='size-[14px]' />
263263
<span className='sr-only'>Add Filter</span>
@@ -269,7 +269,7 @@ export function KnowledgeTagFilters({
269269
removeFilter(filter.id)
270270
}}
271271
disabled={isReadOnly}
272-
className='h-auto p-0'
272+
size='icon'
273273
>
274274
<Trash className='size-[14px]' />
275275
<span className='sr-only'>Delete Filter</span>

‎apps/sim/app/workspace/[workspaceId]/w/[workflowId]/components/panel/components/editor/components/sub-block/components/selector-combobox/selector-combobox.tsx‎

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -263,7 +263,8 @@ export function SelectorCombobox({
263263
<Button
264264
type='button'
265265
variant='ghost'
266-
className='h-auto shrink-0 p-0'
266+
size='icon'
267+
className='shrink-0'
267268
disabled={disabled || readOnly}
268269
aria-label={`Remove ${optionMap.get(id)?.label ?? id}`}
269270
onClick={() => handleMultiChange(selectedValues.filter((v) => v !== id))}

‎apps/sim/app/workspace/[workspaceId]/w/[workflowId]/components/panel/components/editor/components/sub-block/components/sort-builder/components/sort-rule-row.tsx‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -88,15 +88,15 @@ export function SortRuleRow({
8888

8989
const renderActions = () => (
9090
<>
91-
<Button variant='ghost' onClick={onAdd} disabled={isReadOnly} className='h-auto p-0'>
91+
<Button variant='ghost' onClick={onAdd} disabled={isReadOnly} size='icon'>
9292
<Plus className='size-[14px]' />
9393
<span className='sr-only'>Add Sort</span>
9494
</Button>
9595
<Button
9696
variant='ghost-destructive'
9797
onClick={() => onRemove(rule.id)}
9898
disabled={isReadOnly}
99-
className='h-auto p-0'
99+
size='icon'
100100
>
101101
<Trash className='size-[14px]' />
102102
<span className='sr-only'>Delete Sort</span>

‎apps/sim/app/workspace/[workspaceId]/w/[workflowId]/components/panel/components/editor/components/sub-block/components/starter/input-format.tsx‎

Lines changed: 3 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -647,20 +647,16 @@ export function FieldFormat({
647647
animated
648648
actions={
649649
<>
650-
<Button
651-
variant='ghost'
652-
onClick={addField}
653-
disabled={isReadOnly}
654-
className='h-auto p-0'
655-
>
650+
<Button variant='ghost' onClick={addField} disabled={isReadOnly} size='icon'>
656651
<Plus className='size-[14px]' />
657652
<span className='sr-only'>Add {title}</span>
658653
</Button>
659654
<Button
660655
variant='ghost-destructive'
661656
onClick={() => removeField(field.id)}
662657
disabled={isReadOnly}
663-
className='h-auto p-0 hover-hover:opacity-90'
658+
size='icon'
659+
className='hover-hover:opacity-90'
664660
>
665661
<Trash className='size-[14px]' />
666662
<span className='sr-only'>Delete Field</span>

‎apps/sim/app/workspace/[workspaceId]/w/[workflowId]/components/panel/components/editor/components/sub-block/components/variables-input/variables-input.tsx‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -428,7 +428,7 @@ export function VariablesInput({
428428
addAssignment()
429429
}}
430430
disabled={isReadOnly || allVariablesAssigned}
431-
className='h-auto p-0'
431+
size='icon'
432432
>
433433
<Plus className='size-[14px]' />
434434
<span className='sr-only'>Add Variable</span>
@@ -440,7 +440,7 @@ export function VariablesInput({
440440
removeAssignment(assignment.id)
441441
}}
442442
disabled={isReadOnly}
443-
className='h-auto p-0'
443+
size='icon'
444444
>
445445
<Trash className='size-[14px]' />
446446
<span className='sr-only'>Delete Variable</span>

‎apps/sim/app/workspace/[workspaceId]/w/[workflowId]/components/variables/variables.tsx‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -457,7 +457,7 @@ export function Variables({ readOnly = false }: VariablesProps) {
457457
e.stopPropagation()
458458
handleRemoveVariable(variable.id)
459459
}}
460-
className='h-auto p-0'
460+
size='icon'
461461
disabled={readOnly}
462462
aria-label={`Delete ${variable.name || `variable ${index + 1}`}`}
463463
>

0 commit comments

Comments
 (0)