Clean up

adsabs · Oct 25, 2023 · d4788ca · d4788ca
1 parent b5a1862
commit d4788ca
Show file tree

Hide file tree

Showing 2 changed files with 25 additions and 10 deletions.
diff --git a/src/components/Libraries/LibraryEntityPane.tsx b/src/components/Libraries/LibraryEntityPane.tsx
@@ -353,13 +353,19 @@ const BulkAction = ({
   return (
     <Flex justifyContent="space-between" backgroundColor="gray.50" p={4}>
       <HStack gap={2}>
-        <Checkbox
-          size="md"
-          isChecked={isAllSelected}
-          isIndeterminate={!isAllSelected && isSomeSelected}
-          onChange={handleChange}
-          mr={2}
-        />
+        <Tooltip
+          label={
+            isAllSelected || isSomeSelected ? 'clear selected papers on this page' : 'select all papers on this page'
+          }
+        >
+          <Checkbox
+            size="md"
+            isChecked={isAllSelected}
+            isIndeterminate={!isAllSelected && isSomeSelected}
+            onChange={handleChange}
+            mr={2}
+          />
+        </Tooltip>
         {selectedCount > 0 && (
           <>
             <>{selectedCount} selected</>

diff --git a/src/pages/user/libraries/[[...id]].tsx b/src/pages/user/libraries/[[...id]].tsx
@@ -55,9 +55,18 @@ const LibrariesHome: NextPage<ILibrariesHomeProps> = ({ id, subpage }) => {
           }
         />
       )}
-      {!!id && !!library && !subpage && <LibraryEntityPane library={library} publicView={false} onRefetch={refetch} />}
-      {!!id && !!library && subpage === 'settings' && <LibrarySettingsPane library={library} onRefetch={refetch} />}
-      {!id && <LibrariesLandingPane />}
+
+      {!!id && !!library ? (
+        <>
+          {subpage === 'settings' ? (
+            <LibrarySettingsPane library={library} onRefetch={refetch} />
+          ) : (
+            <LibraryEntityPane library={library} publicView={false} onRefetch={refetch} />
+          )}
+        </>
+      ) : (
+        <>{!id ? <LibrariesLandingPane /> : null}</>
+      )}
     </>
   );
 };