From 527e23cab319ca94b1e5ed8f08e88a402de0f4c9 Mon Sep 17 00:00:00 2001
From: Hiroki Terashima <honchikun@gmail.com>
Date: Mon, 13 May 2024 17:39:08 -0700
Subject: [PATCH] refactor(NavItemComponent): Convert to standalone (#1795)

---
 src/app/student/vle/student-vle.module.ts     |   2 +-
 .../vle/nav-item/nav-item.component.html      |   6 +-
 .../wise5/vle/nav-item/nav-item.component.ts  | 103 ++++++++++--------
 3 files changed, 62 insertions(+), 49 deletions(-)

diff --git a/src/app/student/vle/student-vle.module.ts b/src/app/student/vle/student-vle.module.ts
index 6be029ee21a..07214d01fd8 100644
--- a/src/app/student/vle/student-vle.module.ts
+++ b/src/app/student/vle/student-vle.module.ts
@@ -42,7 +42,6 @@ import { NodeStatusIconComponent } from '../../../assets/wise5/themes/default/th
     GenerateImageDialogComponent,
     GroupTabsComponent,
     NavigationComponent,
-    NavItemComponent,
     NodeNavigationComponent,
     RunEndedAndLockedMessageComponent,
     SafeUrl,
@@ -55,6 +54,7 @@ import { NodeStatusIconComponent } from '../../../assets/wise5/themes/default/th
     CommonModule,
     ComponentStudentModule,
     MatDialogModule,
+    NavItemComponent,
     NodeModule,
     NodeStatusIconComponent,
     SimpleDialogModule,
diff --git a/src/assets/wise5/vle/nav-item/nav-item.component.html b/src/assets/wise5/vle/nav-item/nav-item.component.html
index 8620d1fe62f..de2f70763a3 100644
--- a/src/assets/wise5/vle/nav-item/nav-item.component.html
+++ b/src/assets/wise5/vle/nav-item/nav-item.component.html
@@ -1,8 +1,8 @@
 <ng-container *ngIf="nodeStatus.isVisible" [ngSwitch]="type">
   <mat-card *ngSwitchCase="'card'" class="nav-item" [ngClass]="{ expanded: expanded }">
     <mat-card-header
-      (click)="itemClicked($event)"
-      (keyup.enter)="itemClicked($event)"
+      (click)="itemClicked()"
+      (keyup.enter)="itemClicked()"
       role="button"
       matRipple
       fxLayout="row"
@@ -69,7 +69,7 @@
     <button
       mat-list-item
       class="list-item nav-item"
-      (click)="itemClicked($event)"
+      (click)="itemClicked()"
       tabindex="0"
       i18n-title
       title="Go to {{ nodeTitle }}"
diff --git a/src/assets/wise5/vle/nav-item/nav-item.component.ts b/src/assets/wise5/vle/nav-item/nav-item.component.ts
index f9d0469b03d..85dfb62f854 100644
--- a/src/assets/wise5/vle/nav-item/nav-item.component.ts
+++ b/src/assets/wise5/vle/nav-item/nav-item.component.ts
@@ -1,80 +1,93 @@
-'use strict';
-
 import { Component, Input, ViewEncapsulation } from '@angular/core';
 import { Subscription } from 'rxjs';
 import { ProjectService } from '../../services/projectService';
 import { StudentDataService } from '../../services/studentDataService';
 import { NodeService } from '../../services/nodeService';
+import { CommonModule } from '@angular/common';
+import { MatCardModule } from '@angular/material/card';
+import { NodeIconComponent } from '../node-icon/node-icon.component';
+import { FlexLayoutModule } from '@angular/flex-layout';
+import { MatProgressBarModule } from '@angular/material/progress-bar';
+import { NodeStatusIconComponent } from '../../themes/default/themeComponents/nodeStatusIcon/node-status-icon.component';
+import { MatIconModule } from '@angular/material/icon';
+import { MatTooltipModule } from '@angular/material/tooltip';
+import { MatListModule } from '@angular/material/list';
 
 @Component({
+  encapsulation: ViewEncapsulation.None,
+  imports: [
+    CommonModule,
+    FlexLayoutModule,
+    MatCardModule,
+    MatIconModule,
+    MatListModule,
+    MatProgressBarModule,
+    MatTooltipModule,
+    NodeIconComponent,
+    NodeStatusIconComponent
+  ],
   selector: 'nav-item',
-  styleUrls: ['nav-item.component.scss'],
-  templateUrl: 'nav-item.component.html',
-  encapsulation: ViewEncapsulation.None
+  standalone: true,
+  styleUrl: 'nav-item.component.scss',
+  templateUrl: 'nav-item.component.html'
 })
 export class NavItemComponent {
-  currentNode: any;
-  expanded: boolean = false;
-  isCurrentNode: boolean;
-  isGroup: boolean;
-  isPrevStep: boolean = false;
-  item: any;
-  subscriptions: Subscription = new Subscription();
-
-  @Input()
-  nodeId: string;
-  nodeStatus: any;
-  nodeTitle: string;
-
-  @Input()
-  showPosition: any;
-
-  @Input()
-  type: string;
+  private currentNode: any;
+  protected expanded: boolean;
+  private isCurrentNode: boolean;
+  protected isGroup: boolean;
+  protected isPrevStep: boolean;
+  protected item: any;
+  @Input() nodeId: string;
+  protected nodeStatus: any;
+  protected nodeTitle: string;
+  @Input() showPosition: any;
+  private subscriptions: Subscription = new Subscription();
+  @Input() type: string;
 
   constructor(
+    private dataService: StudentDataService,
     private nodeService: NodeService,
-    private ProjectService: ProjectService,
-    private StudentDataService: StudentDataService
+    private projectService: ProjectService
   ) {}
 
-  ngOnInit() {
-    this.item = this.ProjectService.idToNode[this.nodeId];
-    this.isGroup = this.ProjectService.isGroupNode(this.nodeId);
-    this.nodeStatus = this.StudentDataService.nodeStatuses[this.nodeId];
+  ngOnInit(): void {
+    this.item = this.projectService.idToNode[this.nodeId];
+    this.isGroup = this.projectService.isGroupNode(this.nodeId);
+    this.nodeStatus = this.dataService.nodeStatuses[this.nodeId];
     this.nodeTitle = this.showPosition
-      ? this.ProjectService.nodeIdToNumber[this.nodeId] + ': ' + this.item.title
+      ? this.projectService.nodeIdToNumber[this.nodeId] + ': ' + this.item.title
       : this.item.title;
-    this.currentNode = this.StudentDataService.currentNode;
+    this.currentNode = this.dataService.currentNode;
     this.isCurrentNode = this.currentNode.id === this.nodeId;
     if (this.isGroup && this.isCurrentNode) {
       this.setExpanded();
     }
     this.subscriptions.add(
-      this.StudentDataService.navItemIsExpanded$.subscribe(({ nodeId, isExpanded }) => {
+      this.dataService.navItemIsExpanded$.subscribe(({ nodeId, isExpanded }) => {
         if (nodeId === this.nodeId) {
           this.expanded = isExpanded;
         }
       })
     );
     this.subscriptions.add(
-      this.StudentDataService.currentNodeChanged$.subscribe(
+      this.dataService.currentNodeChanged$.subscribe(
         ({ previousNode: oldNode, currentNode: newNode }) => {
           this.currentNode = newNode;
           this.isCurrentNode = this.nodeId === newNode.id;
           let isPrev = false;
-          if (this.ProjectService.isApplicationNode(newNode.id)) {
+          if (this.projectService.isApplicationNode(newNode.id)) {
             return;
           }
           if (oldNode) {
             isPrev = this.nodeId === oldNode.id;
-            if (this.StudentDataService.previousStep) {
-              this.isPrevStep = this.nodeId === this.StudentDataService.previousStep.id;
+            if (this.dataService.previousStep) {
+              this.isPrevStep = this.nodeId === this.dataService.previousStep.id;
             }
           }
           if (this.isGroup) {
-            let prevNodeisGroup = !oldNode || this.ProjectService.isGroupNode(oldNode.id);
-            let prevNodeIsDescendant = this.ProjectService.isNodeDescendentOfGroup(
+            let prevNodeisGroup = !oldNode || this.projectService.isGroupNode(oldNode.id);
+            let prevNodeIsDescendant = this.projectService.isNodeDescendentOfGroup(
               oldNode,
               this.item
             );
@@ -93,7 +106,7 @@ export class NavItemComponent {
               }
             }
           } else {
-            if (isPrev && this.ProjectService.isNodeDescendentOfGroup(this.item, newNode)) {
+            if (isPrev && this.projectService.isNodeDescendentOfGroup(this.item, newNode)) {
               this.zoomToElement();
             }
           }
@@ -102,20 +115,20 @@ export class NavItemComponent {
     );
   }
 
-  ngOnDestroy() {
+  ngOnDestroy(): void {
     this.subscriptions.unsubscribe();
   }
 
-  setExpanded(): void {
+  private setExpanded(): void {
     this.expanded = true;
-    this.StudentDataService.setNavItemExpanded(this.nodeId, this.expanded);
+    this.dataService.setNavItemExpanded(this.nodeId, this.expanded);
   }
 
-  zoomToElement() {
+  private zoomToElement(): void {
     // TODO: implement me
   }
 
-  itemClicked(event) {
+  protected itemClicked(): void {
     if (this.isGroup) {
       this.expanded = !this.expanded;
       if (this.expanded) {
@@ -125,7 +138,7 @@ export class NavItemComponent {
           this.nodeService.setCurrentNode(this.nodeId);
         }
       }
-      this.StudentDataService.setNavItemExpanded(this.nodeId, this.expanded);
+      this.dataService.setNavItemExpanded(this.nodeId, this.expanded);
     } else {
       this.nodeService.setCurrentNode(this.nodeId);
     }