Fixed a scope issue in the detail compoenents' calls to the region se…

…rvice and fixed a syntax error in the track details template.
legumeinfo · Nov 2, 2022 · ec90bb9 · ec90bb9
1 parent 41e11e8
commit ec90bb9
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 6 deletions.
diff --git a/src/app/gene/components/details/gene/gene-detail.component.ts b/src/app/gene/components/details/gene/gene-detail.component.ts
@@ -71,7 +71,7 @@ export class GeneDetailComponent implements OnDestroy, OnInit {
       .pipe(
         takeUntil(this._destroy),
         take(1))
-      .subscribe(this._processGeneLinks);
+      .subscribe((links) => this._processGeneLinks(links));
 
     // get gene region details
     this._geneService.getGenes([this.gene], this.source)
@@ -89,7 +89,7 @@ export class GeneDetailComponent implements OnDestroy, OnInit {
         }),
         takeUntil(this._destroy),
         take(1))
-      .subscribe(this._processRegionLinks);
+      .subscribe((links) => this._processRegionLinks(links));
   }
 
   // private

diff --git a/src/app/gene/components/details/track/track-detail.component.ts b/src/app/gene/components/details/track/track-detail.component.ts
@@ -15,9 +15,11 @@ import { Track } from '@gcv/gene/models';
     <div class="details">
       <h4>{{ track.genus[0] }}.{{ track.species }} - {{ track.name }}</h4>
       <p><a [routerLink]="['/search', track.source, focus]" queryParamsHandling="merge">Search for similar contexts</a></p>
-      <li *ngFor="let link of regionLinks">
-        <a href="{{ link.href }}">{{ link.text }}</a>
-      </li>
+      <ul>
+        <li *ngFor="let link of regionLinks">
+          <a href="{{ link.href }}">{{ link.text }}</a>
+        </li>
+      </ul>
       <p>Genes:</p>
       <ul>
         <li *ngFor="let gene of track.genes; let i = index">
@@ -82,7 +84,7 @@ export class TrackDetailComponent implements OnDestroy, OnInit {
         }),
         takeUntil(this._destroy),
         take(1))
-      .subscribe(this._processRegionLinks);
+      .subscribe((links) => this._processRegionLinks(links));
   }
 
   private _processRegionLinks(links: any[]) {