removed no-people template

Author: nmetulev

src/components/mgt-agenda/mgt-agenda.ts

@@ -256,7 +256,7 @@ export class MgtAgenda extends MgtTemplatedComponent {
       return this.renderTemplate('loading', null) || this.renderLoading();
     }
 
-    if (this.events) {
+    if (this.events && this.events.length) {
       const events = this.showMax && this.showMax > 0 ? this.events.slice(0, this.showMax) : this.events;
 
       const renderedTemplate = this.renderTemplate('default', { events });

src/components/mgt-people/mgt-people.ts

@@ -119,36 +119,32 @@ export class MgtPeople extends MgtTemplatedComponent {
    */
 
   protected render() {
-    if (this.people) {
-      if (this.people.length) {
-        return (
-          this.renderTemplate('default', { people: this.people }) ||
-          html`
-            <ul class="people-list">
-              ${this.people.slice(0, this.showMax).map(
-                person =>
-                  html`
-                    <li class="people-person">
-                      ${this.renderTemplate('person', { person }, person.displayName) || this.renderPerson(person)}
-                    </li>
-                  `
-              )}
-              ${this.people.length > this.showMax
-                ? this.renderTemplate('overflow', {
-                    extra: this.people.length - this.showMax,
-                    max: this.showMax,
-                    people: this.people
-                  }) ||
-                  html`
+    if (this.people && this.people.length) {
+      return (
+        this.renderTemplate('default', { people: this.people }) ||
+        html`
+          <ul class="people-list">
+            ${this.people.slice(0, this.showMax).map(
+              person =>
+                html`
+                  <li class="people-person">
+                    ${this.renderTemplate('person', { person }, person.displayName) || this.renderPerson(person)}
+                  </li>
+                `
+            )}
+            ${this.people.length > this.showMax
+              ? this.renderTemplate('overflow', {
+                  extra: this.people.length - this.showMax,
+                  max: this.showMax,
+                  people: this.people
+                }) ||
+                html`
                     <li class="overflow"><span>+${this.people.length - this.showMax}<span></li>
                   `
-                : null}
-            </ul>
-          `
-        );
-      } else {
-        return this.renderTemplate('no-people', null) || html``;
-      }
+              : null}
+          </ul>
+        `
+      );
     } else {
       return this.renderTemplate('no-data', null) || html``;
     }

src/components/mgt-tasks/mgt-tasks.ts

@@ -1071,7 +1071,7 @@ export class MgtTasks extends MgtTemplatedComponent {
       : [];
 
     const noPeopleTemplate = html`
-      <template data-type="no-people">
+      <template data-type="no-data">
         <i class="login-icon ms-Icon ms-Icon--Contact"></i>
       </template>
     `;