Skip to content

Commit d7b4aff

Browse files
committed
fix(access-control): keep the block disclosure chevron inside its toggle button
Splitting the chevron out so the Info badge could sit beside the name left the chevron with no click handler — the visible expansion affordance did nothing. It goes back inside the button; Info stays outside it, since an Info trigger is itself a button and cannot nest.
1 parent 8d2fa69 commit d7b4aff

1 file changed

Lines changed: 9 additions & 8 deletions

File tree

apps/sim/ee/access-control/components/group-detail.tsx

Lines changed: 9 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -749,20 +749,21 @@ function BlockToolRow({
749749
{deniedCount} blocked
750750
</ChipTag>
751751
)}
752+
{isBlockAllowed && isExpandable && (
753+
<ChevronDown
754+
className={cn(
755+
'ml-auto size-[14px] flex-shrink-0 text-[var(--text-icon)] transition-transform',
756+
expanded && 'rotate-180'
757+
)}
758+
/>
759+
)}
752760
</button>
761+
{/* Outside the button: an Info trigger is itself a button and cannot nest. */}
753762
{block.description && (
754763
<Info side='top' className='flex-shrink-0'>
755764
{block.description}
756765
</Info>
757766
)}
758-
{isBlockAllowed && isExpandable && (
759-
<ChevronDown
760-
className={cn(
761-
'ml-auto size-[14px] flex-shrink-0 text-[var(--text-icon)] transition-transform',
762-
expanded && 'rotate-180'
763-
)}
764-
/>
765-
)}
766767
</div>
767768
{expanded && isBlockAllowed && isExpandable && (
768769
<div className='border-[var(--border)] border-t px-2 pt-2 pb-3'>

0 commit comments

Comments
 (0)