Skip to content

Commit e7fa13b

Browse files
committed
fix(ObjectPage): use UI5WC tabbar shadow
1 parent 05eb6e5 commit e7fa13b

File tree

2 files changed

+7
-12
lines changed

2 files changed

+7
-12
lines changed

packages/main/src/components/ObjectPage/ObjectPage.module.css

Lines changed: 5 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -105,21 +105,15 @@
105105
z-index: 4;
106106
}
107107

108+
.tabContainerSpacer {
109+
height: 3px;
110+
background: var(--sapObjectHeader_Background);
111+
}
112+
108113
.tabContainer {
109114
position: sticky;
110115
z-index: 3;
111116
background: var(--sapObjectHeader_Background);
112-
padding-block-start: 3px;
113-
&::after {
114-
content: '';
115-
pointer-events: none;
116-
position: absolute;
117-
height: 1px;
118-
inset-block-end: -1px;
119-
inset-inline-start: 0;
120-
width: 100%;
121-
box-shadow: var(--sapContent_HeaderShadow);
122-
}
123117
}
124118

125119
.tabContainerComponent {
@@ -130,7 +124,6 @@
130124
&::part(tabstrip) {
131125
padding: 0;
132126
padding-inline: var(--_ui5wcr_ObjectPage_tab_bar_inline_padding);
133-
box-shadow: none;
134127
}
135128
}
136129

packages/main/src/components/ObjectPage/index.tsx

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -726,6 +726,8 @@ const ObjectPage = forwardRef<ObjectPageDomRef, ObjectPagePropTypes>((props, ref
726726
: `${topHeaderHeight}px`,
727727
}}
728728
>
729+
{/* Spacer required to prevent overlapping of tabbar btns and expand/pin btns*/}
730+
<div aria-hidden="true" className={classNames.tabContainerSpacer} />
729731
<TabContainer
730732
collapsed
731733
onTabSelect={handleTabSelect}

0 commit comments

Comments
 (0)