Bug/212 decode errors #229
Conversation
| >{{ company.apprenticeshipManager?.CompanyName }}<br | ||
| /></ng-container> | ||
| <ng-container *ngIf="company.apprenticeshipManager?.CompanyNameAddition" | ||
| >{{ company.apprenticeshipManager?.CompanyNameAddition }}<br |
There was a problem hiding this comment.
Gibt es hier einen zusätzlichen Abstand, wegen dem <br>, wenn nur eines der beiden vornhanden ist, oder ist das kein Problem? Vielleicht könntest du die einzelnen Zeilen einfach in ein <div> wrappen, welches ein Blockelement ist: <div *ngIf="company.apprenticeshipManager?.CompanyName" >{{ company.apprenticeshipContract.CompanyName }}</div>. Das wäre wohl grundsätzlich im ganzen File der Fall.
There was a problem hiding this comment.
Sollte nicht, die
sind ja innerhalb vom Container. Oder verstehe ich etwas falsch?
Im konkreten Beispiel, wo CompanyNameAddition nicht gesetzt ist, gibt es keinen Zusätzlichen Abstand.
Ansonsten müsste man es dann gleich in allen Dateien anpassen, wo wir das bisher so machen...
There was a problem hiding this comment.
Nein, ich denke es ist OK so 👍
| > | ||
| {{ company.jobTrainer.Lastname }} | ||
| {{ company.jobTrainer.Firstname }}<br /> | ||
| </ng-container> |
There was a problem hiding this comment.
Könnte ev. auch nur der Nachname gesetzt sein?
There was a problem hiding this comment.
Ich weiss es nicht. Das scheint mir nicht wahrscheinlich.
Dieselbe Frage würde sich dann auch bei apprenticeshipManager und person stellen.
There was a problem hiding this comment.
Es wäre wohl etwas robuster dies vorzusehen:
<ng-container *ngIf="company.jobTrainer.Lastname || company.jobTrainer.Firstname"
>
{{ company.jobTrainer.Lastname || '' }}
{{ company.jobTrainer.Firstname || '' }}<br />
</ng-container>
Aber ist wohl nicht so wichtig...
There was a problem hiding this comment.
Find ich gut und habe ich für die Fälle in der ApprenticeshipCompany-Komponente so übernommen.
No description provided.