Skip to content

Commit 39de31c

Browse files
refactor(ui5-side-navigation): remove "design" property (SAP#11219)
* refactor(ui5-side-navigation): remove "design" property
1 parent 90819f4 commit 39de31c

File tree

19 files changed

+80
-91
lines changed

19 files changed

+80
-91
lines changed

packages/fiori/src/SideNavigation.ts

Lines changed: 0 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,6 @@ import { isInstanceOfSideNavigationItem } from "./SideNavigationItem.js";
3131
import type SideNavigationItem from "./SideNavigationItem.js";
3232
import type SideNavigationSubItem from "./SideNavigationSubItem.js";
3333
import type SideNavigationGroup from "./SideNavigationGroup.js";
34-
import type SideNavigationDesign from "./types/SideNavigationDesign.js";
3534
import SideNavigationTemplate from "./SideNavigationTemplate.js";
3635

3736
import {
@@ -142,16 +141,6 @@ class SideNavigation extends UI5Element {
142141
@property({ type: Boolean })
143142
collapsed = false;
144143

145-
/**
146-
* Defines whether the control should have container styling or not.
147-
* **Note** In order to achieve the best user experience, it is recommended to use "Plain" value if SideNavigation is placed inside a responsive popover.
148-
*
149-
* @public
150-
* @default "Decorated"
151-
*/
152-
@property()
153-
design: `${SideNavigationDesign}` = "Decorated";
154-
155144
/**
156145
* Defines the accessible ARIA name of the component.
157146
* @default undefined

packages/fiori/src/themes/NavigationLayout.css

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,6 @@
1717
}
1818

1919
.ui5-nl-header {
20-
box-shadow: var(--sapShell_Shadow);
2120
z-index: 2;
2221
}
2322

@@ -41,6 +40,10 @@
4140
width: 100%;
4241
}
4342

43+
:host(:not([is-phone])) ::slotted([ui5-side-navigation][slot="sideContent"]) {
44+
box-shadow: var(--_ui5_nl_side_navigation_box_shadow);
45+
}
46+
4447
:host([is-phone]) ::slotted([ui5-side-navigation][slot="sideContent"]) {
4548
width: 100%;
4649
box-shadow: none;

packages/fiori/src/themes/SideNavigation.css

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,6 @@
77
width: var(--_ui5_side_navigation_width);
88
max-width: 100%;
99
transition: width 0.3s, min-width 0.3s;
10-
box-shadow: var(--_ui5_side_navigation_box_shadow);
1110
font-family: "72override", var(--sapFontFamily);
1211
font-size: var(--sapFontSize);
1312
background: var(--sapList_Background);
@@ -18,12 +17,7 @@
1817
width: var(--_ui5_side_navigation_collapsed_width);
1918
}
2019

21-
/* SideNavigationDesign */
22-
:host([design="Plain"]) {
23-
box-shadow: none;
24-
}
25-
26-
:host([design="Plain"][is-phone]) {
20+
:host([is-phone]) {
2721
width: 100%;
2822
}
2923

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,3 @@
1+
:root {
2+
--_ui5_nl_side_navigation_box_shadow: none;
3+
}

packages/fiori/src/themes/sap_fiori_3/parameters-bundle.css

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@
77
@import "../base/ProductSwitchItem-parameters.css";
88
@import "./ShellBar-parameters.css";
99
@import "../base/SideNavigation-parameters.css";
10+
@import "../base/NavigationLayout-parameters.css";
1011
@import "../base/TimelineItem-parameters.css";
1112
@import "../base/UploadCollection-parameters.css";
1213
@import "../base/ViewSettingsDialog-parameters.css";

packages/fiori/src/themes/sap_fiori_3_dark/parameters-bundle.css

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@
77
@import "../base/ProductSwitchItem-parameters.css";
88
@import "./ShellBar-parameters.css";
99
@import "../base/SideNavigation-parameters.css";
10+
@import "../base/NavigationLayout-parameters.css";
1011
@import "../base/TimelineItem-parameters.css";
1112
@import "../base/UploadCollection-parameters.css";
1213
@import "../base/ViewSettingsDialog-parameters.css";

packages/fiori/src/themes/sap_fiori_3_hcb/parameters-bundle.css

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@
77
@import "./ProductSwitchItem-parameters.css";
88
@import "../base/ShellBar-parameters.css";
99
@import "../base/SideNavigation-parameters.css";
10+
@import "../base/NavigationLayout-parameters.css";
1011
@import "./TimelineItem-parameters.css";
1112
@import "./UploadCollection-parameters.css";
1213
@import "../base/ViewSettingsDialog-parameters.css";

packages/fiori/src/themes/sap_fiori_3_hcw/parameters-bundle.css

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@
77
@import "./ProductSwitchItem-parameters.css";
88
@import "../base/ShellBar-parameters.css";
99
@import "../base/SideNavigation-parameters.css";
10+
@import "../base/NavigationLayout-parameters.css";
1011
@import "./TimelineItem-parameters.css";
1112
@import "./UploadCollection-parameters.css";
1213
@import "../base/ViewSettingsDialog-parameters.css";
Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,3 @@
1+
:root {
2+
--_ui5_nl_side_navigation_box_shadow: var(--sapContent_Shadow0);
3+
}

packages/fiori/src/themes/sap_horizon/parameters-bundle.css

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,7 @@
1010
@import "./ShellBar-parameters.css";
1111
@import "./TimelineItem-parameters.css";
1212
@import "./SideNavigation-parameters.css";
13+
@import "./NavigationLayout-parameters.css";
1314
@import "./UploadCollection-parameters.css";
1415
@import "./Wizard-parameters.css";
1516
@import "./WizardTab-parameters.css";

0 commit comments

Comments
 (0)