-
Notifications
You must be signed in to change notification settings - Fork 39
feat: templated external links #1910
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Changes from 15 commits
a5a3c28
9bc67e1
0e97ce2
004482b
7255794
237d4a1
f263e26
946f145
67671bb
0f7c8e8
3cbeb67
16c4dec
584613f
6b407e3
12c11b8
e9b65e3
f40bd25
f757346
79a6695
469142a
9dd6624
2c9308f
ba107ec
41712c6
f8f5107
71574f7
a9a92e6
a112e25
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -26,7 +26,7 @@ | |
| <div fxLayout="row" fxLayout.xs="column" *ngIf="dataset"> | ||
| <div [fxFlex]="(attachments$ | async)?.length > 0 ? '90%' : '100%'"> | ||
| <mat-card [formGroup]="form" data-cy="general-info"> | ||
| <mat-card-header class="general-header"> | ||
| <mat-card-header class="dataset-general-header"> | ||
| <mat-icon class="section-icon"> description </mat-icon> | ||
| <mat-card-title class="section-title">{{ "General Information" | translate }}</mat-card-title> | ||
| </mat-card-header> | ||
|
|
@@ -165,7 +165,7 @@ | |
| </mat-card> | ||
|
|
||
| <mat-card> | ||
| <mat-card-header class="creator-header"> | ||
| <mat-card-header class="dataset-creator-header"> | ||
| <mat-icon class="section-icon"> person </mat-icon> | ||
| <mat-card-title class="section-title">{{ "Creator Information" | translate }}</mat-card-title> | ||
| </mat-card-header> | ||
|
|
@@ -205,7 +205,7 @@ | |
| </mat-card> | ||
|
|
||
| <mat-card> | ||
| <mat-card-header class="file-header"> | ||
| <mat-card-header class="dataset-file-header"> | ||
| <mat-icon class="section-icon"> folder </mat-icon> | ||
| <mat-card-title class="section-title">{{ "File Information" | translate }}</mat-card-title> | ||
| </mat-card-header> | ||
|
|
@@ -229,7 +229,7 @@ | |
| </mat-card> | ||
|
|
||
| <mat-card> | ||
| <mat-card-header class="related-header"> | ||
| <mat-card-header class="dataset-related-header"> | ||
| <mat-icon class="section-icon"> library_books </mat-icon> | ||
| <mat-card-title class="section-title">{{ "Related Documents" | translate }}</mat-card-title> | ||
| </mat-card-header> | ||
|
|
@@ -319,7 +319,7 @@ | |
| </mat-card> | ||
|
|
||
| <mat-card *ngIf="dataset.type === 'derived'"> | ||
| <mat-card-header class="derived-header"> | ||
| <mat-card-header class="dataset-derived-header"> | ||
| <div mat-card-avatar class="section-icon"> | ||
| <mat-icon> analytics </mat-icon> | ||
| </div> | ||
|
|
@@ -345,10 +345,99 @@ | |
| </mat-card> | ||
|
|
||
| <mat-card> | ||
| <mat-card-header | ||
| class="scientific-header"> | ||
| <mat-icon class="section-icon"> science </mat-icon> | ||
| <mat-card-title class="section-title">{{ "Scientific Metadata" | translate }}</mat-card-title> | ||
| <mat-card-header class="dataset-links-header"> | ||
| <mat-icon class="section-icon"> link </mat-icon> | ||
| <mat-card-title class="section-title">{{ "Links" | translate }}</mat-card-title> | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. question (bug_risk): Translation usage for the new "Links" title is inconsistent with the rest of the template Other headers in this template pass the |
||
| </mat-card-header> | ||
|
|
||
| <mat-card-content> | ||
| <table> | ||
| <tr *ngIf="dataset['proposalId'] && proposal"> | ||
| <th>{{ "Proposal" | translate }}</th> | ||
| <td> | ||
| <a (click)="onClickProposal(proposal.proposalId)">{{ | ||
| proposal.title | ||
| }}</a> | ||
| </td> | ||
| </tr> | ||
| <tr | ||
| *ngIf=" | ||
| dataset['proposalId'] && | ||
| !proposal && | ||
| appConfig.datasetDetailsShowMissingProposalId | ||
| " | ||
| > | ||
| <th>{{ "Proposal Id" | translate }}</th> | ||
| <td>{{ dataset["proposalId"] }}</td> | ||
| </tr> | ||
| <tr *ngIf="dataset['sampleId'] && sample"> | ||
| <th>{{ "Sample" | translate }}</th> | ||
| <td> | ||
| <a (click)="onClickSample(sample.sampleId)"> | ||
| <span>{{ sample.description }}</span> | ||
| </a> | ||
| </td> | ||
| </tr> | ||
| <tr *ngIf="dataset['instrumentId'] && instrument"> | ||
| <th>{{ "Instrument" | translate }}</th> | ||
| <td> | ||
| <a (click)="onClickInstrument(instrument.pid)"> | ||
| {{ instrument.name }} | ||
| </a> | ||
| </td> | ||
| </tr> | ||
| <tr *ngIf="dataset['creationLocation'] as value"> | ||
| <th>{{ "Creation Location" | translate }}</th> | ||
| <td>{{ value }}</td> | ||
| </tr> | ||
| <tr *ngIf="dataset.techniques && dataset.techniques.length > 0"> | ||
| <th>{{ "Techniques" | translate }}</th> | ||
| <td> | ||
| <div *ngFor="let technique of dataset.techniques"> | ||
| <span> | ||
| {{ technique.name }} | ||
| </span> | ||
| <span | ||
| *ngIf=" | ||
| dataset.techniques.indexOf(technique) < | ||
| dataset.techniques.length - 1 | ||
| " | ||
| >, | ||
| </span> | ||
| </div> | ||
| </td> | ||
| </tr> | ||
|
GBirkel marked this conversation as resolved.
Outdated
|
||
| <tr | ||
| *ngIf=" | ||
| dataset['inputDatasets'] && dataset['inputDatasets'].length > 0 | ||
| " | ||
| > | ||
| <th>{{ "Input Datasets" | translate }}</th> | ||
| <td> | ||
| <div *ngFor="let datasetPid of dataset['inputDatasets']"> | ||
| <span> | ||
| <a [routerLink]="['/datasets/', datasetPid]"> | ||
| {{ datasetPid }} | ||
| </a> | ||
| </span> | ||
| <span | ||
| *ngIf=" | ||
| dataset['inputDatasets'].indexOf(datasetPid) < | ||
| dataset['inputDatasets'].length - 1 | ||
| " | ||
| >, | ||
| </span> | ||
| </div> | ||
| </td> | ||
| </tr> | ||
| </table> | ||
| </mat-card-content> | ||
| </mat-card> | ||
|
|
||
| <mat-card> | ||
| <mat-card-header class="dataset-scientific-header"> | ||
| <mat-icon class="section-icon"> science </mat-icon> | ||
| <mat-card-title class="section-title">{{ "Scientific Metadata" | translate }}</mat-card-title> | ||
| </mat-card-header> | ||
| <mat-card-content> | ||
| <ng-template | ||
|
|
||
|
GBirkel marked this conversation as resolved.
|
Uh oh!
There was an error while loading. Please reload this page.