Skip to content

Commit

Permalink
Remove experimental flag for webcomponents (SAP#2622)
Browse files Browse the repository at this point in the history
  • Loading branch information
JohannesDoberer authored Mar 24, 2022
1 parent d6dc8f0 commit 319a7c1
Show file tree
Hide file tree
Showing 5 changed files with 4 additions and 22 deletions.
4 changes: 2 additions & 2 deletions core/src/services/routing.js
Original file line number Diff line number Diff line change
Expand Up @@ -351,13 +351,13 @@ class RoutingClass {
}
}

if (nodeObject.compound && GenericHelpers.requestExperimentalFeature('webcomponents', true)) {
if (nodeObject.compound) {
if (iContainer) {
iContainer.classList.add('lui-webComponent');
}
Iframe.switchActiveIframe(iframeElement, undefined, false);
this.navigateWebComponentCompound(component, nodeObject);
} else if (nodeObject.webcomponent && GenericHelpers.requestExperimentalFeature('webcomponents', true)) {
} else if (nodeObject.webcomponent) {
if (iContainer) {
iContainer.classList.add('lui-webComponent');
}
Expand Down
8 changes: 2 additions & 6 deletions docs/web-component.md
Original file line number Diff line number Diff line change
Expand Up @@ -51,15 +51,11 @@ Luigi.setConfig({
...
]
}
settings: {
...
experimental: {
webcomponents:true
}
.......
})
```

> **NOTE:** With Luigi version NEXTRELEASE we removed the experimental flag for webcomponents.
### Write a Web Component

There are a couple of differences between Luigi Web Components and standard ones:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -632,11 +632,6 @@ describe('Luigi client linkManager', () => {
describe('Webcomponent visibleForFeatureToggles test', () => {
beforeEach(() => {
cy.visitLoggedIn('/projects/pr1/wc_grid');
cy.window().then(win => {
const config = win.Luigi.getConfig();
config.settings.experimental = { webcomponents: true };
win.Luigi.configChanged();
});
});

it('open webcomponent with visibleForFeatureToggles', () => {
Expand All @@ -654,11 +649,6 @@ describe('Luigi client linkManager', () => {
describe('Webcomponent compound view test', () => {
beforeEach(() => {
cy.visitLoggedIn('/projects/pr1/wc_grid_compound');
cy.window().then(win => {
const config = win.Luigi.getConfig();
config.settings.experimental = { webcomponents: true };
win.Luigi.configChanged();
});
});

it('open webcomponent btn', () => {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -65,7 +65,6 @@ class Settings {
// experimental = {
// profileMenuFiori3: true,
// globalNav: true,
// webcomponents: true
// };
// globalSideNavigation = true;
}
Expand Down
3 changes: 0 additions & 3 deletions website/fiddle/src/defaultConfig.js
Original file line number Diff line number Diff line change
Expand Up @@ -188,9 +188,6 @@ Luigi.setConfig({
logo: 'img/luigi.png',
title: 'Luigi Fiddle'
},
experimental: {
webcomponents:true
},
burgerTooltip: {
navExpanded: 'Collapse navigation',
navCollapsed: 'Expand navigation'
Expand Down

0 comments on commit 319a7c1

Please sign in to comment.