fix: remove usage of false in component return

src/modules/item/Item.tsx

@@ -535,7 +535,7 @@ const Item = ({
   id,
   isChildren = false,
   showPinnedOnly = false,
-}: Props): JSX.Element | false => {
+}: Props): JSX.Element | null => {
   const { t: translateMessage } = useMessagesTranslation();
   const { data: item, isInitialLoading: isLoadingItem, isError } = useItem(id);
   const title = usePageTitle();
@@ -584,7 +584,7 @@ const Item = ({
       </Alert>
     );
   }
-  return false;
+  return null;
 };
 
 export default Item;

src/modules/navigationIsland/ChatButton.tsx

@@ -14,7 +14,7 @@ import { PLAYER } from '@/langs/constants';
 
 import { ToolButton } from './CustomButtons';
 
-const useChatButton = (): { chatButton: JSX.Element | false } => {
+const useChatButton = (): { chatButton: JSX.Element | null } => {
   const { t } = usePlayerTranslation();
   const { itemId, rootId } = useParams();
   const { data: item } = hooks.useItem(itemId);
@@ -64,6 +64,6 @@ const useChatButton = (): { chatButton: JSX.Element | false } => {
     };
   }
   // disable the chat button if there are no items with the chat enabled
-  return { chatButton: false };
+  return { chatButton: null };
 };
 export default useChatButton;

src/modules/navigationIsland/GeolocationButton.tsx

@@ -15,7 +15,7 @@ import { ToolButton } from './CustomButtons';
 
 const cm = ClientHostManager.getInstance();
 
-const useGeolocationButton = (): { geolocationButton: JSX.Element | false } => {
+const useGeolocationButton = (): { geolocationButton: JSX.Element | null } => {
   const { t } = usePlayerTranslation();
   // get inherited geoloc
   const { itemId, rootId } = useParams();
@@ -26,7 +26,7 @@ const useGeolocationButton = (): { geolocationButton: JSX.Element | false } => {
   const { data: geoloc } = hooks.useItemGeolocation(item?.id);
 
   if (!allGeoloc?.length) {
-    return { geolocationButton: false };
+    return { geolocationButton: null };
   }
 
   const url = geoloc

src/modules/navigationIsland/NavigationIsland.tsx

@@ -7,7 +7,7 @@ import useGeolocationButton from './GeolocationButton';
 import usePinnedItemsButton from './PinnedItemsButton';
 import usePreviousNextButtons from './PreviousNextButtons';
 
-const NavigationIslandBox = (): JSX.Element | false => {
+const NavigationIslandBox = (): JSX.Element | null => {
   const { previousButton, nextButton } = usePreviousNextButtons();
   const { chatButton } = useChatButton();
   const { geolocationButton } = useGeolocationButton();
@@ -21,7 +21,7 @@ const NavigationIslandBox = (): JSX.Element | false => {
     !nextButton &&
     !geolocationButton
   ) {
-    return false;
+    return null;
   }
 
   return (

src/modules/navigationIsland/PinnedItemsButton.tsx

@@ -14,7 +14,7 @@ import { PLAYER } from '@/langs/constants';
 
 import { ToolButton } from './CustomButtons';
 
-const usePinnedItemsButton = (): { pinnedButton: JSX.Element | false } => {
+const usePinnedItemsButton = (): { pinnedButton: JSX.Element | null } => {
   const { t } = usePlayerTranslation();
   const { togglePinned, isPinnedOpen } = useLayoutContext();
   const { itemId } = useParams();
@@ -29,7 +29,7 @@ const usePinnedItemsButton = (): { pinnedButton: JSX.Element | false } => {
 
   // don't show the button if there are no items pinned in all descendants
   if (childrenPinnedCount <= 0) {
-    return { pinnedButton: false };
+    return { pinnedButton: null };
   }
 
   const canWrite = item?.permission

src/modules/navigationIsland/PreviousNextButtons.tsx

@@ -13,8 +13,8 @@ import { combineUuids, shuffleAllButLastItemInArray } from '@/utils/shuffle.ts';
 import { LoadingButton, NavigationButton } from './CustomButtons';
 
 const usePreviousNextButtons = (): {
-  previousButton: JSX.Element | false;
-  nextButton: JSX.Element | false;
+  previousButton: JSX.Element | null;
+  nextButton: JSX.Element | null;
 } => {
   const { rootId, itemId } = useParams();
   const [searchParams] = useSearchParams();
@@ -52,7 +52,7 @@ const usePreviousNextButtons = (): {
 
   // if there are no descendants then there is no need to navigate
   if (!isArray(descendants)) {
-    return { previousButton: false, nextButton: false };
+    return { previousButton: null, nextButton: null };
   }
 
   let folderHierarchy = descendants;
@@ -78,7 +78,7 @@ const usePreviousNextButtons = (): {
 
     // if index is not found, then do not show navigation
     if (idx < 0) {
-      return { previousButton: false, nextButton: false };
+      return { previousButton: null, nextButton: null };
     }
 
     // if index is 0, previous is root
@@ -101,7 +101,7 @@ const usePreviousNextButtons = (): {
 
   // should we display both buttons if they are disabled ?
   if (!prev && !next) {
-    return { previousButton: false, nextButton: false };
+    return { previousButton: null, nextButton: null };
   }
 
   return {

src/modules/pages/HomePage.tsx

@@ -35,7 +35,7 @@ const DisplayItems = ({
 }: {
   items?: PackedItem[];
   isLoading: boolean;
-}): JSX.Element[] | false => {
+}): JSX.Element[] | null => {
   if (items) {
     return items.map((item) => (
       <GridWrapper key={item.id}>
@@ -50,7 +50,7 @@ const DisplayItems = ({
       </GridWrapper>
     ));
   }
-  return false;
+  return null;
 };
 
 const HomePage = (): JSX.Element => {