From c2b25cbbdd6462876259273cc94e696a6738baeb Mon Sep 17 00:00:00 2001
From: "Doberer, Johannes" <johannes.doberer@sap.com>
Date: Wed, 20 Oct 2021 20:30:13 +0200
Subject: [PATCH 1/4] changeable-config-properties

---
 core/src/Authorization.html                   |  2 +-
 core/src/AuthorizationSimpleProfileMenu.html  | 15 ++++++++----
 core/src/TopNavDropDown.html                  |  2 +-
 core/src/navigation/ContextSwitcher.html      |  4 ++--
 core/src/navigation/GlobalNav.html            |  3 ++-
 core/src/navigation/LogoTitle.html            | 19 ++++++++-------
 core/src/navigation/ProductSwitcher.html      |  6 ++---
 core/src/navigation/TopNav.html               | 23 +++++++++++++++----
 .../src/luigi-config/extended/settings.js     | 10 ++++----
 9 files changed, 52 insertions(+), 32 deletions(-)

diff --git a/core/src/Authorization.html b/core/src/Authorization.html
index caeb3bb7a0..9affdb4781 100644
--- a/core/src/Authorization.html
+++ b/core/src/Authorization.html
@@ -154,6 +154,7 @@
   const dispatch = createEventDispatcher();
 
   export let isHidden = false;
+  export let addNavHrefForAnchor;
 
   let idpProviderInstance;
   let userInfo;
@@ -169,7 +170,6 @@
   let getUnsavedChangesModalPromise = getContext('getUnsavedChangesModalPromise');
   let openViewInModal = getContext('openViewInModal');
   let initialsOfUser;
-  const addNavHrefForAnchor = LuigiConfig.getConfigBooleanValue('navigation.addNavHrefs');
 
   onMount(async () => {
     if (!LuigiAuth.isAuthorizationEnabled()) {
diff --git a/core/src/AuthorizationSimpleProfileMenu.html b/core/src/AuthorizationSimpleProfileMenu.html
index 884304849b..0973aeb4ac 100644
--- a/core/src/AuthorizationSimpleProfileMenu.html
+++ b/core/src/AuthorizationSimpleProfileMenu.html
@@ -37,13 +37,18 @@
     </li>
     {/each} {#if hasUserSettings}
     <li
-      tabindex="-1" 
+      tabindex="-1"
       class="fd-menu__item lui-anchor-node"
       on:click|preventDefault="{onUserSettingsClick}"
       on:keyup="{(event) => handleKeyUp(event)}"
       data-testid="{getTestId(profileNav.settings)}"
     >
-      <a tabindex="0" title="User Settings" class="fd-menu__link" data-testid="settings-link">
+      <a
+        tabindex="0"
+        title="User Settings"
+        class="fd-menu__link"
+        data-testid="settings-link"
+      >
         {#if profileNav.settings.icon} {#if hasOpenUIicon(profileNav.settings)}
         <i class="fd-top-nav__icon sap-icon--{profileNav.settings.icon}"></i>
         {:else}
@@ -104,6 +109,7 @@
   const dispatch = createEventDispatcher();
 
   export let isHidden = false;
+  export let addNavHrefForAnchor;
 
   let idpProviderInstance;
   let userInfo;
@@ -119,7 +125,6 @@
   let getUnsavedChangesModalPromise = getContext('getUnsavedChangesModalPromise');
   let openViewInModal = getContext('openViewInModal');
   let initialsOfUser;
-  const addNavHrefForAnchor = LuigiConfig.getConfigBooleanValue('navigation.addNavHrefs');
 
   onMount(async () => {
     if (!LuigiAuth.isAuthorizationEnabled()) {
@@ -296,10 +301,10 @@
   }
 
   li > button.fd-menu__link {
-    background-color: var(--sapList_Background,#fff);
+    background-color: var(--sapList_Background, #fff);
 
     &:hover {
-      background-color: var(--sapList_Hover_Background,#f5f5f5);
+      background-color: var(--sapList_Hover_Background, #f5f5f5);
     }
   }
 </style>
diff --git a/core/src/TopNavDropDown.html b/core/src/TopNavDropDown.html
index 89109b2e2f..cbd31037ac 100644
--- a/core/src/TopNavDropDown.html
+++ b/core/src/TopNavDropDown.html
@@ -62,7 +62,7 @@
   let pathParams;
   let getUnsavedChangesModalPromise = getContext('getUnsavedChangesModalPromise');
   let openViewInModal = getContext('openViewInModal');
-  const addNavHrefForAnchor = LuigiConfig.getConfigBooleanValue('navigation.addNavHrefs');
+  let addNavHrefForAnchor;
   const getNodeSubtitle = () => {};
 
   //TODO refactor
diff --git a/core/src/navigation/ContextSwitcher.html b/core/src/navigation/ContextSwitcher.html
index 424516b74e..83184dfe89 100644
--- a/core/src/navigation/ContextSwitcher.html
+++ b/core/src/navigation/ContextSwitcher.html
@@ -4,8 +4,7 @@
 <div class="fd-shellbar__action fd-shellbar__action--desktop">
   <div class="fd-popover fd-popover--right">
     <div class="fd-popover__control" on:click|stopPropagation="{()=> {}}">
-      {#if LuigiConfig.getConfigBooleanValue('navigation.addNavHrefs') && (selectedOption
-      !== config.defaultLabel) }
+      {#if addNavHrefForAnchor && (selectedOption !== config.defaultLabel) }
       <a
         href="{selectedNodePath}"
         class="fd-button fd-button--transparent fd-button--menu fd-shellbar__button--menu lui-ctx-switch-menu"
@@ -160,6 +159,7 @@ <h2 class="fd-title fd-title--h5" id="dialog-title-3">
   let getTranslation = getContext('getTranslation');
   let prevContextSwitcherToggle = false;
   let selectedNodePath;
+  let addNavHrefForAnchor;
 
   onMount(async () => {
     StateHelpers.doOnStoreChange(
diff --git a/core/src/navigation/GlobalNav.html b/core/src/navigation/GlobalNav.html
index 8466031339..551f283ec5 100644
--- a/core/src/navigation/GlobalNav.html
+++ b/core/src/navigation/GlobalNav.html
@@ -104,7 +104,7 @@
   export let showGlobalNav;
   export let hideNavComponent;
   export let responsiveNavSetting;
-  const addNavHrefForAnchor = LuigiConfig.getConfigBooleanValue('navigation.addNavHrefs');
+  let addNavHrefForAnchor = LuigiConfig.getConfigBooleanValue('navigation.addNavHrefs');
 
   const setTopNavData = async () => {
     if (pathData && 0 < pathData.length) {
@@ -136,6 +136,7 @@
     if (!previousPathData || previousPathData != pathData) {
       setTopNavData();
     }
+    addNavHrefForAnchor = LuigiConfig.getConfigBooleanValue('navigation.addNavHrefs');
   });
 
   function hasOpenUIicon(node) {
diff --git a/core/src/navigation/LogoTitle.html b/core/src/navigation/LogoTitle.html
index 27771e25a3..0677958b55 100644
--- a/core/src/navigation/LogoTitle.html
+++ b/core/src/navigation/LogoTitle.html
@@ -1,4 +1,4 @@
-{#if LuigiConfig.getConfigBooleanValue('navigation.addNavHrefs')}
+{#if addNavHrefForAnchor}
 <a
   class="fd-shellbar__logo { !hasLogo ? 'fd-shellbar__logo--image-replaced' : '' } { hasLogo ? 'lui-customlogo' : ''}"
   aria-label="{title}"
@@ -23,8 +23,7 @@
   <img data-testid="luigi-topnav-logo" bind:this="{logo}" alt="{title}" />
   {/if}
 </span>
-{/if} {#if title} {#if !hasApps} {#if
-LuigiConfig.getConfigBooleanValue('navigation.addNavHrefs')}
+{/if} {#if title} {#if !hasApps} {#if addNavHrefForAnchor}
 <a
   class="fd-shellbar__title lui-shellbar-single-app-title"
   data-testid="luigi-topnav-title"
@@ -44,11 +43,11 @@
 {/if}{:else}
 <div class="fd-popover">
   <div class="fd-popover__control" on:click|stopPropagation="{() => {}}">
-    {#if LuigiConfig.getConfigBooleanValue('navigation.addNavHrefs')} {#if
-    appSwitcherItems && appSwitcherItems.length > 0} {#each appSwitcherItems as item} {#if
-    item === selectedItem && hasValidLink(item, pathParams)}
+    {#if addNavHrefForAnchor} {#if appSwitcherItems && appSwitcherItems.length > 0} {#each
+    appSwitcherItems as item} {#if item === selectedItem && hasValidLink(item,
+    pathParams)}
     <a
-      href="{addNavHrefForAnchor ? getRouteLink(item) : undefined}"
+      href="{getRouteLink(item)}"
       class="fd-button fd-button--transparent fd-button--menu fd-shellbar__button--menu lui-app-switch"
       aria-haspopup="true"
       aria-expanded="{dropDownStates.appSwitcherPopover || false}"
@@ -60,7 +59,7 @@
       >
       <i class="sap-icon sap-icon--megamenu fd-shellbar__button--icon"></i>
     </a>
-    {/if}{/each}{/if} {#if appSwitcherItems.length > 0 && !selectedItem}
+    {/if} {/each} {/if} {#if appSwitcherItems.length > 0 && !selectedItem}
     <a
       href="/"
       class="fd-button fd-button--transparent fd-button--menu fd-shellbar__button--menu lui-app-switch"
@@ -134,7 +133,7 @@
   import { beforeUpdate, createEventDispatcher, onMount, getContext } from 'svelte';
   import * as Header from './services/header';
   import { Routing } from '../services/routing';
-  import { NavigationHelpers, RoutingHelpers } from '../utilities/helpers';
+  import { NavigationHelpers, RoutingHelpers, StateHelpers } from '../utilities/helpers';
   import { LuigiConfig } from '../core-api';
 
   const dispatch = createEventDispatcher();
@@ -152,11 +151,11 @@
   export let subTitle;
   export let defaultSubTitle;
   export let pathData;
+  export let addNavHrefForAnchor;
   let previousPathData;
   let getUnsavedChangesModalPromise = getContext('getUnsavedChangesModalPromise');
   let getTranslation = getContext('getTranslation');
   let store = getContext('store');
-  const addNavHrefForAnchor = LuigiConfig.getConfigBooleanValue('navigation.addNavHrefs');
 
   //TODO refactor
   const getComponentWrapper = () => {
diff --git a/core/src/navigation/ProductSwitcher.html b/core/src/navigation/ProductSwitcher.html
index 2d89e883ac..5a35112471 100644
--- a/core/src/navigation/ProductSwitcher.html
+++ b/core/src/navigation/ProductSwitcher.html
@@ -39,9 +39,9 @@
               on:click="{() => onActionClick(productSwitcherItem)}"
               data-testid="{getTestId(productSwitcherItem)}"
             >
-              {#if LuigiConfig.getConfigBooleanValue('navigation.addNavHrefs')}
+              {#if addNavHrefForAnchor}
               <a
-                href="{addNavHrefForAnchor ? getRouteLink(productSwitcherItem) : undefined}"
+                href="{getRouteLink(productSwitcherItem)}"
                 on:click|preventDefault="{() => {}}"
                 class="fd-menu__link"
               >
@@ -131,7 +131,7 @@
   let store = getContext('store');
   let getUnsavedChangesModalPromise = getContext('getUnsavedChangesModalPromise');
   let columnsClass;
-  const addNavHrefForAnchor = LuigiConfig.getConfigBooleanValue('navigation.addNavHrefs');
+  export let addNavHrefForAnchor;
 
   onMount(async () => {
     StateHelpers.doOnStoreChange(
diff --git a/core/src/navigation/TopNav.html b/core/src/navigation/TopNav.html
index 35d2cebd1e..363a7eaa27 100644
--- a/core/src/navigation/TopNav.html
+++ b/core/src/navigation/TopNav.html
@@ -16,6 +16,7 @@
     <LogoTitle
       pathData="{pathData}"
       pathParams="{pathParams}"
+      addNavHrefForAnchor="{addNavHrefForAnchor}"
       bind:dropDownStates
       on:toggleDropdownState="{toggleDropdownStateExternal}"
       on:handleClick="{handleClickExternal}"
@@ -39,6 +40,7 @@
       bind:dropDownStates
       on:toggleDropdownState="{() => toggleDropdownState('contextSwitcherPopover')}"
       isMobile="{false}"
+      addNavHrefForAnchor="{addNavHrefForAnchor}"
     />
     {/if} {#if isGlobalSearchAvailable || (children && pathData.length > 0)} {#if
     (children && pathData.length > 0)} {#each children as node, i} {#if !(node.hideFromNav
@@ -66,7 +68,12 @@
           aria-hidden="{!(dropDownStates[`dropDownPopover-${i}`] || false)}"
           id="dropDownPopover-{i}"
         >
-          <TopNavDropDown node="{node}" isMobile="{false}" pathParams="{pathParams}" />
+          <TopNavDropDown
+            node="{node}"
+            isMobile="{false}"
+            pathParams="{pathParams}"
+            addNavHrefForAnchor="{addNavHrefForAnchor}"
+          />
         </div>
       </div>
     </div>
@@ -74,9 +81,9 @@
     <div
       class="fd-shellbar__action fd-shellbar__action--hide fd-shellbar__action--desktop"
     >
-      {#if LuigiConfig.getConfigBooleanValue('navigation.addNavHrefs')}
+      {#if addNavHrefForAnchor}
       <a
-        href="{addNavHrefForAnchor ? getRouteLink(node) : undefined}"
+        href="{getRouteLink(node)}"
         class="fd-shellbar__button fd-button {node === selectedNode ? 'is-selected' : ''}"
         aria-expanded="false"
         aria-haspopup="true"
@@ -330,6 +337,7 @@
           <Authorization
             on:toggleDropdownState="{() => toggleDropdownState('profilePopover')}"
             on:userInfoUpdated="{userInfoUpdate}"
+            addNavHrefForAnchor="{addNavHrefForAnchor}"
           />
         </div>
       </div>
@@ -369,6 +377,7 @@
             <AuthorizationSimpleProfileMenu
               on:toggleDropdownState="{() => toggleDropdownState('profilePopover')}"
               on:userInfoUpdated="{userInfoUpdate}"
+              addNavHrefForAnchor="{addNavHrefForAnchor}"
             />
           </div>
         </div>
@@ -380,6 +389,7 @@
       bind:dropDownStates
       on:toggleDropdownState="{() => toggleDropdownState('productSwitcherPopover')}"
       isMobile="{false}"
+      addNavHrefForAnchor="{addNavHrefForAnchor}"
     />
     {/if}
   </div>
@@ -388,6 +398,7 @@
 <AuthorizationSimpleProfileMenu
   on:toggleDropdownState="{() => toggleDropdownState('profilePopover')}"
   isHidden="{true}"
+  addNavHrefForAnchor="{addNavHrefForAnchor}"
 />
 {/if}
 <script>
@@ -451,7 +462,9 @@
   let selectedLabel;
   let defaultLabelContextSwitcher;
   let contextSwitcherConfig = LuigiConfig.getConfigValue('navigation.contextSwitcher');
-  const addNavHrefForAnchor = LuigiConfig.getConfigBooleanValue('navigation.addNavHrefs');
+  export let addNavHrefForAnchor = LuigiConfig.getConfigBooleanValue(
+    'navigation.addNavHrefs'
+  );
   const setTopNavData = async () => {
     if (pathData && 0 < pathData.length) {
       const tnd = await NavigationHelpers.generateTopNavNodes(pathData);
@@ -489,6 +502,8 @@
         showGlobalNav =
           LuigiConfig.getConfigBooleanValue('settings.globalSideNavigation') &&
           GenericHelpers.requestExperimentalFeature('globalNav', true);
+        addNavHrefForAnchor = LuigiConfig.getConfigBooleanValue('navigation.addNavHrefs');
+        contextSwitcherConfig = LuigiConfig.getConfigValue('navigation.contextSwitcher');
       },
       ['navigation']
     );
diff --git a/test/e2e-test-application/src/luigi-config/extended/settings.js b/test/e2e-test-application/src/luigi-config/extended/settings.js
index 02aa35795d..a9e16ee2da 100644
--- a/test/e2e-test-application/src/luigi-config/extended/settings.js
+++ b/test/e2e-test-application/src/luigi-config/extended/settings.js
@@ -57,11 +57,11 @@ class Settings {
   //   };
   // };
   // globalSideNavigation = true;
-  // experimental = {
-  //    profileMenuFiori3: true,
-  // globalNav: true
-  //   webcomponents:true
-  // };
+  experimental = {
+    profileMenuFiori3: true,
+    globalNav: true,
+    webcomponents: true
+  };
   // globalSideNavigation = true;
 }
 

From e4bda04161a04aa10974c586f7c1631ed9aafd38 Mon Sep 17 00:00:00 2001
From: "Doberer, Johannes" <johannes.doberer@sap.com>
Date: Wed, 20 Oct 2021 21:43:15 +0200
Subject: [PATCH 2/4] sideNavCompactMode

---
 core/src/navigation/LeftNav.html | 12 ++++--------
 1 file changed, 4 insertions(+), 8 deletions(-)

diff --git a/core/src/navigation/LeftNav.html b/core/src/navigation/LeftNav.html
index 78b9f5960a..63da4af5ce 100644
--- a/core/src/navigation/LeftNav.html
+++ b/core/src/navigation/LeftNav.html
@@ -48,7 +48,9 @@
     <div class="fd-side-nav__main-navigation">
       {#if children && pathData.length > 1}
       <div class="lui-fd-side-nav-wrapper">
-        <ul class="fd-nested-list {sideNavClass}">
+        <ul
+          class="fd-nested-list {sideNavCompactMode ? 'fd-nested-list fd-nested-list--compact':'fd-nested-list'}"
+        >
           {#each sortedChildrenEntries as [key, nodes],index} {#if key === 'undefined' ||
           key.startsWith(virtualGroupPrefix)}
           <!-- Single nodes -->
@@ -452,7 +454,6 @@
   export let selectedCategory = null;
   export let expandedCategories;
   export let hasCategoriesWithIcon;
-  export let sideNavClass;
   export let sideNavAccordionMode;
   export let burgerTooltip;
   export let navHeader;
@@ -493,12 +494,6 @@
       ['settings.footer']
     );
 
-    // set compact mode class
-    if (Boolean(sideNavCompactMode)) {
-      sideNavClass = 'fd-nested-list fd-nested-list--compact';
-    } else {
-      sideNavClass = 'fd-nested-list';
-    }
     let stateArr = SemiCollapsibleNavigation.initial();
     isSemiCollapsed = stateArr.isSemiCollapsed;
     semiCollapsible = stateArr.semiCollapsible;
@@ -517,6 +512,7 @@
     if (!previousPathData || previousPathData != pathData) {
       setLeftNavData();
     }
+    sideNavCompactMode = LuigiConfig.getConfigBooleanValue('settings.sideNavCompactMode');
   });
 
   export let sortedChildrenEntries;

From b727145eb8711e87b6fe3bc78cbf771a5902e062 Mon Sep 17 00:00:00 2001
From: "Doberer, Johannes" <johannes.doberer@sap.com>
Date: Thu, 21 Oct 2021 20:25:31 +0200
Subject: [PATCH 3/4] settings

---
 core/src/navigation/LeftNav.html | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/core/src/navigation/LeftNav.html b/core/src/navigation/LeftNav.html
index 63da4af5ce..91b418831a 100644
--- a/core/src/navigation/LeftNav.html
+++ b/core/src/navigation/LeftNav.html
@@ -513,6 +513,8 @@
       setLeftNavData();
     }
     sideNavCompactMode = LuigiConfig.getConfigBooleanValue('settings.sideNavCompactMode');
+    semiCollapsibleButton =
+      LuigiConfig.getConfigValue('settings.responsiveNavigation') === 'semiCollapsible';
   });
 
   export let sortedChildrenEntries;

From 7f5043564a1ba293807f15e418889e567d0d907f Mon Sep 17 00:00:00 2001
From: "Doberer, Johannes" <johannes.doberer@sap.com>
Date: Thu, 28 Oct 2021 14:06:12 +0200
Subject: [PATCH 4/4] no message

---
 .../src/luigi-config/extended/settings.js              | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/test/e2e-test-application/src/luigi-config/extended/settings.js b/test/e2e-test-application/src/luigi-config/extended/settings.js
index e1661c1c28..c610039a04 100644
--- a/test/e2e-test-application/src/luigi-config/extended/settings.js
+++ b/test/e2e-test-application/src/luigi-config/extended/settings.js
@@ -62,11 +62,11 @@ class Settings {
   //   };
   // };
   // globalSideNavigation = true;
-  experimental = {
-    profileMenuFiori3: true,
-    globalNav: true,
-    webcomponents: true
-  };
+  // experimental = {
+  //   profileMenuFiori3: true,
+  //   globalNav: true,
+  //   webcomponents: true
+  // };
   // globalSideNavigation = true;
 }