From 67dc71b9639c8f8497ae44225287f6680390b8cc Mon Sep 17 00:00:00 2001
From: Lukas Harbarth <lukas.harbarth@sap.com>
Date: Tue, 12 May 2026 13:53:30 +0200
Subject: [PATCH 1/2] fix(ObjectPage): prevent header collapse on IconTabBar
 tab switch

---
 .../components/ObjectPage/ObjectPage.cy.tsx   | 44 ++++++++++++++-----
 .../main/src/components/ObjectPage/index.tsx  | 12 ++++-
 .../components/ObjectPageSection/index.tsx    |  1 +
 3 files changed, 43 insertions(+), 14 deletions(-)

diff --git a/packages/main/src/components/ObjectPage/ObjectPage.cy.tsx b/packages/main/src/components/ObjectPage/ObjectPage.cy.tsx
index 337c892819c..ead0f5d8839 100644
--- a/packages/main/src/components/ObjectPage/ObjectPage.cy.tsx
+++ b/packages/main/src/components/ObjectPage/ObjectPage.cy.tsx
@@ -114,7 +114,7 @@ describe('ObjectPage', () => {
       cy.get('[ui5-tabcontainer]').findUi5TabByText('Section 15').should('have.attr', 'aria-selected', 'true');
 
       if (mode === ObjectPageMode.Default) {
-        cy.findByTestId('op').scrollTo(0, 4858);
+        cy.findByTestId('op').scrollTo(0, 4878);
 
         cy.findByText('Content 7').should('be.visible');
         cy.get('[ui5-tabcontainer]').findUi5TabByText('Section 7').should('have.attr', 'aria-selected', 'true');
@@ -124,7 +124,7 @@ describe('ObjectPage', () => {
         for (let i = 0; i < 15; i++) {
           cy.findByText('Add').click();
         }
-        cy.findByTestId('op').scrollTo(0, 4858);
+        cy.findByTestId('op').scrollTo(0, 4878);
 
         cy.findByText('Content 7').should('be.visible');
         cy.get('[ui5-tabcontainer]').findUi5TabByText('Section 7').should('have.attr', 'aria-selected', 'true');
@@ -754,11 +754,11 @@ describe('ObjectPage', () => {
 
     cy.mount(<TestSingleSectionComp height="2000px" withFooter mode={ObjectPageMode.Default} />);
     cy.findByText('Update Heights').click();
-    cy.findByText('{"offset":1080,"scroll":2320}').should('exist');
+    cy.findByText('{"offset":1080,"scroll":2330}').should('exist');
 
     cy.findByTestId('op').scrollTo('bottom');
     cy.findByText('Update Heights').click({ force: true });
-    cy.findByText('{"offset":1080,"scroll":2320}').should('exist');
+    cy.findByText('{"offset":1080,"scroll":2330}').should('exist');
 
     cy.mount(<TestSingleSectionComp height="400px" mode={ObjectPageMode.Default} />);
     cy.findByText('Update Heights').click();
@@ -812,19 +812,19 @@ describe('ObjectPage', () => {
     cy.get('[data-component-name="ObjectPageTabContainer"]').should('not.exist');
 
     cy.findByText('Update Heights').click();
-    cy.findByText('{"offset":1080,"scroll":2240}').should('exist');
+    cy.findByText('{"offset":1080,"scroll":2250}').should('exist');
 
     cy.findByTestId('op').scrollTo('bottom');
     cy.findByText('Update Heights').click({ force: true });
-    cy.findByText('{"offset":1080,"scroll":2240}').should('exist');
+    cy.findByText('{"offset":1080,"scroll":2250}').should('exist');
 
     cy.mount(<TestSingleSectionComp height="2000px" withFooter mode={ObjectPageMode.IconTabBar} />);
     cy.findByText('Update Heights').click();
-    cy.findByText('{"offset":1080,"scroll":2300}').should('exist');
+    cy.findByText('{"offset":1080,"scroll":2310}').should('exist');
 
     cy.findByTestId('op').scrollTo('bottom');
     cy.findByText('Update Heights').click({ force: true });
-    cy.findByText('{"offset":1080,"scroll":2300}').should('exist');
+    cy.findByText('{"offset":1080,"scroll":2310}').should('exist');
 
     cy.mount(<TestSingleSectionComp height="400px" mode={ObjectPageMode.IconTabBar} />);
     cy.findByText('Update Heights').click();
@@ -1299,6 +1299,13 @@ describe('ObjectPage', () => {
             >
               Select Goals
             </button>
+            <button
+              onClick={() => {
+                setSelectedSection(arbitraryCharsId);
+              }}
+            >
+              Select Employment
+            </button>
             <button
               onClick={() => {
                 setSelectedSubSection('personal-payment-information');
@@ -1349,7 +1356,9 @@ describe('ObjectPage', () => {
           cy.wait(100);
           cy.findByText('Select Goals').click();
           cy.findByText('goals-content').should('be.visible');
-          cy.get('@change').should('have.callCount', callCount);
+          if (mode !== 'IconTabBar') {
+            cy.get('@change').should('have.callCount', callCount);
+          }
           callCount++;
           if (mode === 'IconTabBar') {
             cy.findByText('personal-connect-content').should('not.exist');
@@ -1366,7 +1375,9 @@ describe('ObjectPage', () => {
           }
           cy.findByText('personal-connect-content').should('be.visible');
           cy.get('[ui5-tabcontainer]').findUi5TabByText('Personal').should('have.attr', 'aria-selected', 'true');
-          cy.get('@change').should('have.callCount', callCount);
+          if (mode !== 'IconTabBar') {
+            cy.get('@change').should('have.callCount', callCount);
+          }
           callCount++;
 
           cy.get('[ui5-tabcontainer]').findUi5TabByText('Goals').click();
@@ -1377,14 +1388,23 @@ describe('ObjectPage', () => {
           }
           cy.findByText('goals-content').should('be.visible');
           cy.get('[ui5-tabcontainer]').findUi5TabByText('Goals').should('have.attr', 'aria-selected', 'true');
-          cy.get('@change').should('have.callCount', callCount);
+          if (mode !== 'IconTabBar') {
+            cy.get('@change').should('have.callCount', callCount);
+          }
           callCount++;
 
+          // Re-visiting a section should not collapse the header
+          if (mode === 'IconTabBar' && props.headerArea) {
+            cy.get('[data-component-name="ObjectPageHeaderContent"]').should('be.visible');
+          }
+
           cy.findByText('Select Payment Information').click();
           cy.findByText('personal-payment-information-content').should('be.visible');
           cy.findByText('personal-connect-content').should('not.be.visible');
           cy.get('[ui5-tabcontainer]').findUi5TabByText('Personal').should('have.attr', 'aria-selected', 'true');
-          cy.get('@change').should('have.callCount', callCount);
+          if (mode !== 'IconTabBar') {
+            cy.get('@change').should('have.callCount', callCount);
+          }
           callCount++;
         },
       );
diff --git a/packages/main/src/components/ObjectPage/index.tsx b/packages/main/src/components/ObjectPage/index.tsx
index 5878a6a4565..5968256170c 100644
--- a/packages/main/src/components/ObjectPage/index.tsx
+++ b/packages/main/src/components/ObjectPage/index.tsx
@@ -317,11 +317,16 @@ const ObjectPage = forwardRef<ObjectPageDomRef, ObjectPagePropTypes>((props, ref
     debouncedOnSectionChange.cancel();
     setSelectedSubSectionId(undefined);
     setInternalSelectedSectionId((prevSelectedSection) => {
-      if (prevSelectedSection === newSelectionSectionId) {
+      if (prevSelectedSection === newSelectionSectionId && mode !== ObjectPageMode.IconTabBar) {
         scrollToSection(newSelectionSectionId);
       }
       return newSelectionSectionId;
     });
+    // Reset scroll for section swap; scrollTimeout preserves current header collapsed/expanded state.
+    if (mode === ObjectPageMode.IconTabBar) {
+      scrollTimeout.current = performance.now() + 500;
+      objectPageRef.current?.scrollTo({ top: 1 });
+    }
     setTabSelectId(newSelectionSectionId);
     scrollEvent.current = targetEvent;
     if (isMounted && mode === ObjectPageMode.Default) {
@@ -344,6 +349,7 @@ const ObjectPage = forwardRef<ObjectPageDomRef, ObjectPagePropTypes>((props, ref
         }
       };
       if (mode === ObjectPageMode.IconTabBar) {
+        scrollTimeout.current = performance.now() + 500;
         setInternalSelectedSectionId(selectedSectionId);
         getSectionElementById(objectPageContentRef.current, false, selectedSectionId)?.focus({
           preventScroll: true,
@@ -890,7 +896,9 @@ const ObjectPage = forwardRef<ObjectPageDomRef, ObjectPagePropTypes>((props, ref
           <div
             style={{
               height:
-                ((headerCollapsed && !headerPinned) || scrolledHeaderExpanded) && !toggledCollapsedHeaderWasVisible
+                ((headerCollapsed && !headerPinned) || scrolledHeaderExpanded) &&
+                !toggledCollapsedHeaderWasVisible &&
+                !(mode === ObjectPageMode.IconTabBar && scrollTimeout.current >= performance.now())
                   ? `${headerContentHeight}px`
                   : 0,
             }}
diff --git a/packages/main/src/components/ObjectPageSection/index.tsx b/packages/main/src/components/ObjectPageSection/index.tsx
index 8b3424e5950..f83c8fe475d 100644
--- a/packages/main/src/components/ObjectPageSection/index.tsx
+++ b/packages/main/src/components/ObjectPageSection/index.tsx
@@ -265,6 +265,7 @@ const ObjectPageSection = forwardRef<HTMLElement, ObjectPageSectionPropTypes>((p
           {children}
         </div>
       </div>
+      <div className={classNames.outlineSpacerDiv} aria-hidden="true" />
     </section>
   );
 });

From 105773e35cb22ff3908c68536c1a280108f1351a Mon Sep 17 00:00:00 2001
From: Lukas Harbarth <lukas.harbarth@sap.com>
Date: Tue, 12 May 2026 14:11:53 +0200
Subject: [PATCH 2/2] Update index.tsx

---
 packages/main/src/components/ObjectPage/index.tsx | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/packages/main/src/components/ObjectPage/index.tsx b/packages/main/src/components/ObjectPage/index.tsx
index 5968256170c..c980801409a 100644
--- a/packages/main/src/components/ObjectPage/index.tsx
+++ b/packages/main/src/components/ObjectPage/index.tsx
@@ -325,7 +325,7 @@ const ObjectPage = forwardRef<ObjectPageDomRef, ObjectPagePropTypes>((props, ref
     // Reset scroll for section swap; scrollTimeout preserves current header collapsed/expanded state.
     if (mode === ObjectPageMode.IconTabBar) {
       scrollTimeout.current = performance.now() + 500;
-      objectPageRef.current?.scrollTo({ top: 1 });
+      objectPageRef.current?.scrollTo({ top: 0 });
     }
     setTabSelectId(newSelectionSectionId);
     scrollEvent.current = targetEvent;