Merge pull request #23933 from overleaf/jel-group-table-no-checkbox

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

GitOrigin-RevId: dbbfe31fd8e3a3b9a1ec9fd11d7a8950c7344f83

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">

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>
   )
 }