fix: Stay in all-namespaces after clicking on list item (kyma-project#3807)

* fix: set namespace to -all- in url for navgiation of generic list

* fix: temporary fix for allNamespaces refresh

* fix: typo

Author: chriskari

src/resources/createResourceRoutes.js

@@ -58,7 +58,8 @@ const ColumnWrapper = ({ list, details, create, ...props }) => {
     resourceName: resourceName,
   });
 
-  const defaultColumn = resourceName ? 'details' : 'list';
+  const defaultColumn =
+    resourceName && namespaceId !== '-all-' ? 'details' : 'list';
 
   const layoutCloseCreateUrl = resourceListUrl({
     kind: props.resourceType,

src/shared/components/GenericList/GenericList.js

@@ -310,11 +310,12 @@ export const GenericList = ({
     isResourceEditedState,
   );
   const [isFormOpen, setIsFormOpen] = useRecoilState(isFormOpenState);
-  const { resourceUrl: resourceUrlFn } = useUrl();
+  const { resourceUrl: resourceUrlFn, namespace } = useUrl();
   const linkTo = entry => {
+    const overrides = namespace === '-all-' ? { namespace } : {};
     return customUrl
       ? customUrl(entry)
-      : resourceUrlFn(entry, { resourceType });
+      : resourceUrlFn(entry, { resourceType, ...overrides });
   };
 
   const handleRowClick = e => {

src/shared/hooks/usePrepareLayout.js

@@ -100,7 +100,8 @@ export function usePrepareLayoutColumns({
   const navigationType = useNavigationType();
 
   const newLayoutState = useMemo(() => {
-    if (!layout || layout === 'OneColumn') {
+    const isAllNamespaces = namespaceId === '-all-';
+    if (!layout || layout === 'OneColumn' || isAllNamespaces) {
       return {
         layout: 'OneColumn',
         startColumn: {
@@ -221,7 +222,9 @@ export function usePrepareLayoutColumns({
   useEffect(() => {
     setLayoutColumn(newLayoutState);
     setIsFormOpen({
-      formOpen: !!newLayoutState.showCreate || !!newLayoutState.showEdit,
+      formOpen:
+        namespaceId !== '-all-' &&
+        (!!newLayoutState.showCreate || !!newLayoutState.showEdit),
     });
     // eslint-disable-next-line react-hooks/exhaustive-deps
   }, []);

src/sidebar/SidebarNavigation.tsx

@@ -138,6 +138,10 @@ export function SidebarNavigation() {
                     isFormOpen,
                     setIsFormOpen,
                     () => {
+                      const newNamespace =
+                        e.target.value === t('navigation.all-namespaces')
+                          ? '-all-'
+                          : e.target.value;
                       setLayoutColumn(prevState => ({
                         startColumn: {
                           resourceType:
@@ -146,23 +150,17 @@ export function SidebarNavigation() {
                             prevState.startColumn?.resourceName ?? null,
                           apiGroup: prevState.startColumn?.apiGroup ?? null,
                           apiVersion: prevState.startColumn?.apiVersion ?? null,
-                          namespaceId: e.target.value,
+                          namespaceId: newNamespace,
                         },
                         midColumn: null,
                         endColumn: null,
                         layout: 'OneColumn',
                       }));
-                      return e.target.value === t('navigation.all-namespaces')
-                        ? navigate(
-                            namespaceUrl(resourceType, {
-                              namespace: '-all-',
-                            }),
-                          )
-                        : navigate(
-                            namespaceUrl(resourceType, {
-                              namespace: e.target.value ?? undefined,
-                            }),
-                          );
+                      return navigate(
+                        namespaceUrl(resourceType, {
+                          namespace: newNamespace,
+                        }),
+                      );
                     },
                   );
                 }}