Feature/189 incidents #201
Conversation
| color: rgba($body-color, 0.5); | ||
| padding-right: $btn-padding-x; | ||
| grid-area: comment; | ||
| grid-area: incident; | ||
| display: flex; | ||
| } |
There was a problem hiding this comment.
Ein visuelles Detail: ich würde mit negativen Margins den Button so ausrichten, dass die Elemente schön in einer Linie sind (wie beim Statusbutton, dort haben wir es auch so gemacht). Vielleicht käme es auch gut, wenn du ihn noch etwas weiter nach unten nimmst (unterer Rand Foto).
There was a problem hiding this comment.
Perfekt. Auf der Listenansicht bei kleinen Screens könntest du es auch noch korrigieren:
...app/presence-control/components/presence-control-entry/presence-control-entry.component.html
Show resolved
Hide resolved
src/app/presence-control/components/presence-control-entry/presence-control-entry.component.ts
Outdated
Show resolved
Hide resolved
...esence-control/components/presence-control-incident/presence-control-incident.component.scss
Show resolved
Hide resolved
caebr
force-pushed
the
feature/189-incidents
branch
from
d992d59 to
393b12d
Compare
There was a problem hiding this comment.
In der OpenAbsencesDetailComponent wird noch der Kommentar angezeigt. Ich denke fachlich ist es nicht möglich, dass eine offene Absenz einen Vorfall hat (da sie ja einen anderen PresenceType hat). Wahrscheinlich können wir den Kommentar in diesem Fall dort einfach löschen?
| <span class="incident">{{ | ||
| entry.presenceType?.Designation | ||
| }}</span> | ||
| }}</ng-container> |
There was a problem hiding this comment.
So wie ich es verstehe, auch wenn der Status gewechselt werden kann (= grün oder gelb oder offene Absenz) der Vorfall eben angezeigt werden, nicht? Sonst würden ja Vorfälle gar nicht angezeigt werden? Vielleicht müsste man den settings.absencePresenceTypeId (die Standardabwesenheit, wenn kein Grund gesetzt ist) rausfiltern.
| }}</span> | ||
| <span *ngIf="!entry.presenceType?.IsIncident"> | ||
| {{ 'presence-control.entry.incident' | translate }}</span | ||
| > |
There was a problem hiding this comment.
Würde hier nicht eine Condition reichen:
<span>{{
entry.presenceType?.IsIncident && entry.presenceType?.Designation || 'presence-control.entry.incident'
| translate
}}</span>
No description provided.