fix: display filter button on the overview page for non-variants with…

… one group(#18975) In mobile resolution
SAP · Jun 18, 2024 · 6f22b90 · 6f22b90
1 parent aa6a86c
commit 6f22b90
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 3 deletions.
diff --git a/...ased/components/overview-filter-button/configurator-overview-filter-button.component.html b/...ased/components/overview-filter-button/configurator-overview-filter-button.component.html
@@ -6,7 +6,7 @@
 >
  <ng-container
  *ngIf="
- !(isDisplayOnlyVariant() | async) &&
+ !(isDisplayOnlyVariant() | async) ||
  configurationWithOv.overview.possibleGroups?.length !== 1
  "
  >

diff --git a/...d/components/overview-filter-button/configurator-overview-filter-button.component.spec.ts b/...d/components/overview-filter-button/configurator-overview-filter-button.component.spec.ts
@@ -143,9 +143,21 @@ describe('ConfigurationOverviewFilterButtonComponent', () => {
  expect(htmlElem.classList.contains('ghost')).toBeFalsy();
  });
 
- it('should render no filter button for variant', () => {
+ it('should render filter button for variant in case there is more than one group', () => {
  isDisplayOnlyVariant = true;
- ovConfig.overview.possibleGroups = [];
+ fixture.detectChanges();
+
+ CommonConfiguratorTestUtilsService.expectElementPresent(
+ expect,
+ htmlElem,
+ '.cx-config-filter-button'
+ );
+ });
+
+ it('should render no filter button for variant in case there is only one group', () => {
+ isDisplayOnlyVariant = true;
+ ovConfig.overview.possibleGroups =
+ ovConfig.overview.possibleGroups.slice(1);
  fixture.detectChanges();
 
  CommonConfiguratorTestUtilsService.expectElementNotPresent(