add placeholder buttons to make all rows the same height

This commit is contained in:
Milo Schwartz
2024-11-24 23:28:38 -05:00
parent 6abc80aa7a
commit 2312258468
2 changed files with 19 additions and 2 deletions

View File

@@ -65,6 +65,14 @@ export default function UsersTable({ roles: r }: RolesTableProps) {
return (
<>
<div className="flex items-center justify-end">
{roleRow.isAdmin && (
<Button
variant="ghost"
className="h-8 w-8 p-0 opacity-0 cursor-default"
>
Placeholder
</Button>
)}
{!roleRow.isAdmin && (
<DropdownMenu>
<DropdownMenuTrigger asChild>
@@ -117,7 +125,9 @@ export default function UsersTable({ roles: r }: RolesTableProps) {
roleToDelete={roleToRemove}
afterDelete={() => {
setRoles((prev) =>
prev.filter((r) => r.roleId !== roleToRemove.roleId)
prev.filter(
(r) => r.roleId !== roleToRemove.roleId,
),
);
setUserToRemove(null);
}}

View File

@@ -99,7 +99,9 @@ export default function UsersTable({ users: u }: UsersTableProps) {
return (
<div className="flex flex-row items-center gap-1">
{userRow.isOwner && <Crown className="w-4 h-4 text-yellow-600" />}
{userRow.isOwner && (
<Crown className="w-4 h-4 text-yellow-600" />
)}
<span>{userRow.role}</span>
</div>
);
@@ -113,6 +115,11 @@ export default function UsersTable({ users: u }: UsersTableProps) {
return (
<>
<div className="flex items-center justify-end">
{userRow.isOwner && (
<Button variant="ghost" className="opacity-0 cursor-default">
Placeholder
</Button>
)}
{!userRow.isOwner && (
<>
<DropdownMenu>