From 788f6569d814e2c76f1638fab74d0ba00cad0eac Mon Sep 17 00:00:00 2001
From: ilkin-overleaf <100852799+ilkin-overleaf@users.noreply.github.com>
Date: Thu, 27 Feb 2025 11:13:23 +0200
Subject: [PATCH] Merge pull request #23933 from
 overleaf/jel-group-table-no-checkbox

[web] Fix group table layout when all users are managed

GitOrigin-RevId: dbbfe31fd8e3a3b9a1ec9fd11d7a8950c7344f83
---
 .../components/members-table/members-list.tsx |  4 +-
 .../members-table/select-all-checkbox.tsx     | 46 ++++++++++---------
 2 files changed, 25 insertions(+), 25 deletions(-)

diff --git a/services/web/frontend/js/features/group-management/components/members-table/members-list.tsx b/services/web/frontend/js/features/group-management/components/members-table/members-list.tsx
index 617a85bba1..6fdc81bb25 100644
--- a/services/web/frontend/js/features/group-management/components/members-table/members-list.tsx
+++ b/services/web/frontend/js/features/group-management/components/members-table/members-list.tsx
@@ -54,9 +54,7 @@ export default function MembersList({ groupId }: ManagedUsersListProps) {
       >
         <thead>
           <tr>
-            <th className="cell-checkbox">
-              <SelectAllCheckbox />
-            </th>
+            <SelectAllCheckbox />
             <th className="cell-email">{t('email')}</th>
             <th className="cell-name">{t('name')}</th>
             <th className="cell-last-active">
diff --git a/services/web/frontend/js/features/group-management/components/members-table/select-all-checkbox.tsx b/services/web/frontend/js/features/group-management/components/members-table/select-all-checkbox.tsx
index 85e23f0a4a..9286ce1a2e 100644
--- a/services/web/frontend/js/features/group-management/components/members-table/select-all-checkbox.tsx
+++ b/services/web/frontend/js/features/group-management/components/members-table/select-all-checkbox.tsx
@@ -30,27 +30,29 @@ export default function SelectAllCheckbox() {
   }
 
   return (
-    <BootstrapVersionSwitcher
-      bs3={
-        <input
-          className="select-all"
-          type="checkbox"
-          autoComplete="off"
-          onChange={handleSelectAllNonManagedClick}
-          checked={selectedUsers.length === nonManagedUsers.length}
-          aria-label={t('select_all')}
-          data-testid="select-all-checkbox"
-        />
-      }
-      bs5={
-        <OLFormCheckbox
-          autoComplete="off"
-          onChange={handleSelectAllNonManagedClick}
-          checked={selectedUsers.length === nonManagedUsers.length}
-          aria-label={t('select_all')}
-          data-testid="select-all-checkbox"
-        />
-      }
-    />
+    <th className="cell-checkbox">
+      <BootstrapVersionSwitcher
+        bs3={
+          <input
+            className="select-all"
+            type="checkbox"
+            autoComplete="off"
+            onChange={handleSelectAllNonManagedClick}
+            checked={selectedUsers.length === nonManagedUsers.length}
+            aria-label={t('select_all')}
+            data-testid="select-all-checkbox"
+          />
+        }
+        bs5={
+          <OLFormCheckbox
+            autoComplete="off"
+            onChange={handleSelectAllNonManagedClick}
+            checked={selectedUsers.length === nonManagedUsers.length}
+            aria-label={t('select_all')}
+            data-testid="select-all-checkbox"
+          />
+        }
+      />
+    </th>
   )
 }