style(notification-container): Remove unnecessary white space (#204)

- Fix class set on wrong element causing unnecessary white space Closes #113.
fast-facts · Mar 11, 2021 · 29e3c03 · 29e3c03
1 parent 8a2a0eb
commit 29e3c03
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 13 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
@@ -1,5 +1,5 @@
-<ul>
-  <li class="notifier__container-list" *ngFor="let notification of notifications; trackBy: identifyNotification">
+<ul class="notifier__container-list">
+  <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
@@ -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);