Fix issue to add first paragraph on mobile

paulscherrerinstitute · Apr 4, 2024 · ebb7594 · ebb7594
1 parent 36fa6f1
commit ebb7594
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/scilog/src/app/logbook/widgets/logbook-item/logbook-item.component.spec.ts b/scilog/src/app/logbook/widgets/logbook-item/logbook-item.component.spec.ts
@@ -682,4 +682,10 @@ describe('LogbookItemComponent', () => {
     expect(component.mobile).toEqual(true);
   });
 
+  it('should open dialog with or without last', () => {
+    const dialogOpenMock = spyOn(component.dialog, 'open');
+    component.openDialog();
+    expect(dialogOpenMock.calls.mostRecent().args[1].data['defaultTags']).toEqual(undefined);
+  });
+
 });
diff --git a/scilog/src/app/logbook/widgets/logbook-item/logbook-item.component.ts b/scilog/src/app/logbook/widgets/logbook-item/logbook-item.component.ts
@@ -642,7 +642,7 @@ export class LogbookItemComponent implements OnInit {
     const dialogConfig = new MatDialogConfig();
     dialogConfig.autoFocus = true;
     dialogConfig.disableClose = true;
-    dialogConfig.data = { "defaultTags": this.childSnippets.last.snippet.tags, "config": this.config }
+    dialogConfig.data = { "defaultTags": this.childSnippets?.last?.snippet?.tags, "config": this.config }
     const dialogRef = this.dialog.open(AddContentComponent, dialogConfig);
   }