fix: add geolocation button back (#752)

graasp · Jun 21, 2024 · 4b72b5b · 4b72b5b
1 parent 9879a51
commit 4b72b5b
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 9 deletions.
diff --git a/cypress/e2e/island.cy.ts b/cypress/e2e/island.cy.ts
@@ -52,7 +52,7 @@ describe('Island', () => {
     });
 
     // temporarily disable
-    it.skip('Show map button for parent geolocation', () => {
+    it('Show map button for parent geolocation', () => {
       const parentItem = PackedFolderItemFactory({ settings: {} });
       const child = PackedFolderItemFactory({ parentItem, settings: {} });
       const geolocation = { lat: 0, lng: 0, item: parentItem };
@@ -85,7 +85,7 @@ describe('Island', () => {
     });
 
     // temporarily disable
-    it.skip('Show map button for child geolocation', () => {
+    it('Show map button for child geolocation', () => {
       const parentItem = PackedFolderItemFactory({ settings: {} });
       const child1 = PackedFolderItemFactory({ parentItem, settings: {} });
       const child2 = PackedFolderItemFactory({ parentItem, settings: {} });

diff --git a/src/modules/navigationIsland/NavigationIsland.tsx b/src/modules/navigationIsland/NavigationIsland.tsx
@@ -3,23 +3,23 @@ import { Box, Stack } from '@mui/material';
 import { NAVIGATION_ISLAND_CY } from '@/config/selectors';
 
 import useChatButton from './ChatButton';
-// import useGeolocationButton from './GeolocationButton';
+import useGeolocationButton from './GeolocationButton';
 import usePinnedItemsButton from './PinnedItemsButton';
 import usePreviousNextButtons from './PreviousNextButtons';
 
 const NavigationIslandBox = (): JSX.Element | false => {
   const { previousButton, nextButton } = usePreviousNextButtons();
   const { chatButton } = useChatButton();
-  // const { geolocationButton } = useGeolocationButton();
+  const { geolocationButton } = useGeolocationButton();
   const { pinnedButton } = usePinnedItemsButton();
 
   // if all buttons are disabled do not show the island at all
   if (
     !chatButton &&
     !pinnedButton &&
     !previousButton &&
-    !nextButton
-    // && !geolocationButton
+    !nextButton &&
+    !geolocationButton
   ) {
     return false;
   }
@@ -56,12 +56,11 @@ const NavigationIslandBox = (): JSX.Element | false => {
         )}
         {
           // if one of the button is present, show the stack
-          (chatButton || pinnedButton) && (
-            // || geolocationButton
+          (chatButton || pinnedButton || geolocationButton) && (
             <Stack direction="row" gap={1}>
               {chatButton}
               {pinnedButton}
-              {/* {geolocationButton} */}
+              {geolocationButton}
             </Stack>
           )
         }