From d0d567d1d07eb9cdcc128b6a5d59c78af7eae014 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Dominique=20M=C3=BCller?= <dominique.m.mueller@gmail.com>
Date: Fri, 12 Mar 2021 00:30:03 +0100
Subject: [PATCH 1/2] Fix class set on wrong element causing unnecessary white
 space

---
 .../src/lib/components/notifier-container.component.html      | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/projects/angular-notifier/src/lib/components/notifier-container.component.html b/projects/angular-notifier/src/lib/components/notifier-container.component.html
index 562910c7..e332529f 100644
--- a/projects/angular-notifier/src/lib/components/notifier-container.component.html
+++ b/projects/angular-notifier/src/lib/components/notifier-container.component.html
@@ -1,5 +1,5 @@
-<ul>
-  <li class="notifier__container-list" *ngFor="let notification of notifications; trackBy: identifyNotification">
+<ul class="notifier__container-list">
+  <li *ngFor="let notification of notifications; trackBy: identifyNotification">
     <notifier-notification [notification]="notification" (ready)="onNotificationReady($event)" (dismiss)="onNotificationDismiss($event)">
     </notifier-notification>
   </li>

From 4375b34b4950c90cff2cb9e05ffe650fa3d4c5aa Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Dominique=20M=C3=BCller?= <dominique.m.mueller@gmail.com>
Date: Fri, 12 Mar 2021 00:34:48 +0100
Subject: [PATCH 2/2] Fix unit tests

---
 .../notifier-container.component.html         |  2 +-
 .../notifier-container.component.spec.ts      | 22 +++++++++----------
 2 files changed, 12 insertions(+), 12 deletions(-)

diff --git a/projects/angular-notifier/src/lib/components/notifier-container.component.html b/projects/angular-notifier/src/lib/components/notifier-container.component.html
index e332529f..065c28b5 100644
--- a/projects/angular-notifier/src/lib/components/notifier-container.component.html
+++ b/projects/angular-notifier/src/lib/components/notifier-container.component.html
@@ -1,5 +1,5 @@
 <ul class="notifier__container-list">
-  <li *ngFor="let notification of notifications; trackBy: identifyNotification">
+  <li class="notifier__container-list-item" *ngFor="let notification of notifications; trackBy: identifyNotification">
     <notifier-notification [notification]="notification" (ready)="onNotificationReady($event)" (dismiss)="onNotificationDismiss($event)">
     </notifier-notification>
   </li>
diff --git a/projects/angular-notifier/src/lib/components/notifier-container.component.spec.ts b/projects/angular-notifier/src/lib/components/notifier-container.component.spec.ts
index 698e95d2..3be42db4 100644
--- a/projects/angular-notifier/src/lib/components/notifier-container.component.spec.ts
+++ b/projects/angular-notifier/src/lib/components/notifier-container.component.spec.ts
@@ -61,7 +61,7 @@ describe('Notifier Container Component', () => {
         type: 'SHOW',
       });
       componentFixture.detectChanges();
-      const listElements: Array<DebugElement> = componentFixture.debugElement.queryAll(By.css('.notifier__container-list'));
+      const listElements: Array<DebugElement> = componentFixture.debugElement.queryAll(By.css('.notifier__container-list-item'));
 
       expect(listElements.length).toBe(1);
 
@@ -353,7 +353,7 @@ describe('Notifier Container Component', () => {
       tick();
       componentFixture.detectChanges(); // Run a second change detection (to update the template)
 
-      const listElements: Array<DebugElement> = componentFixture.debugElement.queryAll(By.css('.notifier__container-list'));
+      const listElements: Array<DebugElement> = componentFixture.debugElement.queryAll(By.css('.notifier__container-list-item'));
 
       expect(listElements.length).toBe(0);
       expect(mockNotificationComponent.hide).toHaveBeenCalled();
@@ -397,7 +397,7 @@ describe('Notifier Container Component', () => {
       tick();
       componentFixture.detectChanges(); // Run a second change detection (to update the template)
 
-      const listElements: Array<DebugElement> = componentFixture.debugElement.queryAll(By.css('.notifier__container-list'));
+      const listElements: Array<DebugElement> = componentFixture.debugElement.queryAll(By.css('.notifier__container-list-item'));
 
       expect(listElements.length).toBe(1);
       expect(mockNotificationComponent.hide).not.toHaveBeenCalled();
@@ -454,7 +454,7 @@ describe('Notifier Container Component', () => {
       tick();
       componentFixture.detectChanges(); // Run a second change detection (to update the template)
 
-      const listElements: Array<DebugElement> = componentFixture.debugElement.queryAll(By.css('.notifier__container-list'));
+      const listElements: Array<DebugElement> = componentFixture.debugElement.queryAll(By.css('.notifier__container-list-item'));
 
       const expectedCallTimes = 3;
       expect(listElements.length).toBe(1);
@@ -512,7 +512,7 @@ describe('Notifier Container Component', () => {
       tick(testNotifierConfig.animations.hide.speed);
       componentFixture.detectChanges(); // Run a second change detection (to update the template)
 
-      const listElements: Array<DebugElement> = componentFixture.debugElement.queryAll(By.css('.notifier__container-list'));
+      const listElements: Array<DebugElement> = componentFixture.debugElement.queryAll(By.css('.notifier__container-list-item'));
 
       const expectedCallTimes = 3;
       expect(listElements.length).toBe(1);
@@ -566,7 +566,7 @@ describe('Notifier Container Component', () => {
       tick(testNotifierConfig.animations.hide.speed - <number>testNotifierConfig.animations.overlap);
       componentFixture.detectChanges(); // Run a second change detection (to update the template)
 
-      const listElements: Array<DebugElement> = componentFixture.debugElement.queryAll(By.css('.notifier__container-list'));
+      const listElements: Array<DebugElement> = componentFixture.debugElement.queryAll(By.css('.notifier__container-list-item'));
 
       const expectedCallTimes = 3;
       expect(listElements.length).toBe(1);
@@ -607,7 +607,7 @@ describe('Notifier Container Component', () => {
       tick();
       componentFixture.detectChanges();
 
-      const listElements: Array<DebugElement> = componentFixture.debugElement.queryAll(By.css('.notifier__container-list'));
+      const listElements: Array<DebugElement> = componentFixture.debugElement.queryAll(By.css('.notifier__container-list-item'));
 
       const expectedCallTimes = 2;
       expect(listElements.length).toBe(0);
@@ -665,7 +665,7 @@ describe('Notifier Container Component', () => {
       tick();
       componentFixture.detectChanges(); // Run a second change detection (to update the template)
 
-      const listElements: Array<DebugElement> = componentFixture.debugElement.queryAll(By.css('.notifier__container-list'));
+      const listElements: Array<DebugElement> = componentFixture.debugElement.queryAll(By.css('.notifier__container-list-item'));
 
       const expectedCallTimes = 3;
       expect(listElements.length).toBe(1);
@@ -744,7 +744,7 @@ describe('Notifier Container Component', () => {
       tick();
       componentFixture.detectChanges(); // Run a second change detection (to update the template)
 
-      const listElements: Array<DebugElement> = componentFixture.debugElement.queryAll(By.css('.notifier__container-list'));
+      const listElements: Array<DebugElement> = componentFixture.debugElement.queryAll(By.css('.notifier__container-list-item'));
 
       const expectedCallTimes = 3;
       expect(listElements.length).toBe(1);
@@ -824,7 +824,7 @@ describe('Notifier Container Component', () => {
       tick();
       componentFixture.detectChanges(); // Run a second change detection (to update the template)
 
-      const listElements: Array<DebugElement> = componentFixture.debugElement.queryAll(By.css('.notifier__container-list'));
+      const listElements: Array<DebugElement> = componentFixture.debugElement.queryAll(By.css('.notifier__container-list-item'));
 
       const expectedCallTimes = 3;
       expect(listElements.length).toBe(0);
@@ -876,7 +876,7 @@ describe('Notifier Container Component', () => {
       tick(testNotifierConfig.animations.hide.speed + numberOfNotifications * <number>testNotifierConfig.animations.hide.offset);
       componentFixture.detectChanges(); // Run a second change detection (to update the template)
 
-      const listElements: Array<DebugElement> = componentFixture.debugElement.queryAll(By.css('.notifier__container-list'));
+      const listElements: Array<DebugElement> = componentFixture.debugElement.queryAll(By.css('.notifier__container-list-item'));
 
       const expectedCallTimes = 3;
       expect(listElements.length).toBe(0);