Extracted chooseBackgroundImage to parent. Fixed typo label.textString -> label.text. #2895

Author: hirokiterashima

src/main/webapp/wise5/authoringTool/components/component-authoring.component.ts

@@ -108,6 +108,16 @@ export abstract class ComponentAuthoring {
     });
   }
 
+  chooseBackgroundImage(): void {
+    const params = {
+      isPopup: true,
+      nodeId: this.nodeId,
+      componentId: this.componentId,
+      target: 'background'
+    };
+    this.openAssetChooser(params);
+  }
+
   openAssetChooser(params: any): any {
     return this.ProjectAssetService.openAssetChooser(params).then((data: any) => {
       return this.assetSelected(data);

src/main/webapp/wise5/components/conceptMap/concept-map-authoring/concept-map-authoring.component.ts

@@ -156,16 +156,6 @@ export class ConceptMapAuthoring extends ComponentAuthoring {
     }
   }
 
-  chooseBackgroundImage(): void {
-    const params = {
-      isPopup: true,
-      nodeId: this.nodeId,
-      componentId: this.componentId,
-      target: 'background'
-    };
-    this.openAssetChooser(params);
-  }
-
   chooseNodeImage(conceptMapNodeId: string): void {
     const params = {
       isPopup: true,

src/main/webapp/wise5/components/draw/draw-authoring/draw-authoring.component.ts

@@ -176,16 +176,6 @@ export class DrawAuthoring extends ComponentAuthoring {
     this.componentChanged();
   }
 
-  chooseBackgroundImage(): void {
-    const params = {
-      isPopup: true,
-      nodeId: this.nodeId,
-      componentId: this.componentId,
-      target: 'background'
-    };
-    this.openAssetChooser(params);
-  }
-
   chooseStampImage(stampIndex: number): void {
     const params = {
       isPopup: true,

src/main/webapp/wise5/components/label/label-authoring/label-authoring.component.html

@@ -99,7 +99,7 @@
     <span i18n class="starter-labels-button-label">Starter Labels</span>
     <button mat-raised-button
         color="primary"
-        (click)="addLabelClicked()"
+        (click)="addLabel()"
         matTooltip="Add Starter Label"
         matTooltipPosition="above"
         i18n-matTooltip
@@ -199,7 +199,7 @@
       <span fxFlex></span>
       <button mat-raised-button
           color="primary"
-          (click)="deleteLabelClicked(labelIndex, label)"
+          (click)="deleteLabel(labelIndex, label)"
           matTooltip="Delete Label"
           matTooltipPosition="above"
           i18n-matTooltip
@@ -233,4 +233,4 @@
       i18n-aria-label>
     <mat-icon>delete_sweep</mat-icon>
   </button>
-</div>
+</div>

src/main/webapp/wise5/components/label/label-authoring/label-authoring.component.ts

@@ -59,7 +59,7 @@ export class LabelAuthoring extends ComponentAuthoring {
     this.textInputChangeSubscription.unsubscribe();
   }
 
-  addLabelClicked(): void {
+  addLabel(): void {
     const newLabel = {
       text: $localize`Enter text here`,
       color: 'blue',
@@ -74,23 +74,13 @@ export class LabelAuthoring extends ComponentAuthoring {
     this.componentChanged();
   }
 
-  deleteLabelClicked(index: number, label: any): void {
-    if (confirm($localize`Are you sure you want to delete this label?\n\n${label.textString}`)) {
+  deleteLabel(index: number, label: any): void {
+    if (confirm($localize`Are you sure you want to delete this label?\n\n${label.text}`)) {
       this.authoringComponentContent.labels.splice(index, 1);
       this.componentChanged();
     }
   }
 
-  chooseBackgroundImage(): void {
-    const params = {
-      isPopup: true,
-      nodeId: this.nodeId,
-      componentId: this.componentId,
-      target: 'background'
-    };
-    this.openAssetChooser(params);
-  }
-
   assetSelected({ nodeId, componentId, assetItem, target }): void {
     super.assetSelected({ nodeId, componentId, assetItem, target });
     const fileName = assetItem.fileName;