Skip to content

Commit b89647f

Browse files
committed
More improvements to styling.
1 parent 7a5e28a commit b89647f

2 files changed

Lines changed: 16 additions & 11 deletions

File tree

eform-client/src/app/plugins/modules/time-planning-pn/components/plannings/time-plannings-table/time-plannings-table.component.html

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -86,7 +86,7 @@
8686
</div>
8787
</div>
8888
</ng-container>
89-
<div class="plan-text" *ngIf="row.planningPrDayModels[col.field]?.workerComment !== null">
89+
<div class="plan-text" *ngIf="row.planningPrDayModels[col.field]?.workerComment !== null && row.planningPrDayModels[col.field]?.workerComment !== ''">
9090
<div class="{{ getCellTextColor(row, col.field) }}">
9191
<strong>{{ 'CommentWorker' | translate }}:</strong> {{row.planningPrDayModels[col.field]?.workerComment}}
9292
</div>

eform-client/src/app/plugins/modules/time-planning-pn/components/plannings/time-plannings-table/time-plannings-table.component.ts

Lines changed: 15 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -113,9 +113,10 @@ export class TimePlanningsTableComponent implements OnInit, OnChanges {
113113
try {
114114
const planHours = row.planningPrDayModels[field]?.planHours;
115115
const plannedStarted = row.planningPrDayModels[field]?.plannedStartOfShift1;
116-
const workDayStarted = row.planningPrDayModels[field]?.workDayStarted;
116+
const workDayStarted = row.planningPrDayModels[field]?.start1StartedAt || row.planningPrDayModels[field]?.start2StartedAt;
117117
const workDayEnded = row.planningPrDayModels[field]?.workDayEnded;
118118
const message = row.planningPrDayModels[field]?.message;
119+
const workerComment = row.planningPrDayModels[field]?.workerComment;
119120
if (planHours > 0) {
120121
if (workDayStarted) {
121122
//console.log('getCellClass', row, field, planHours, workDayStarted, workDayEnded);
@@ -126,7 +127,7 @@ export class TimePlanningsTableComponent implements OnInit, OnChanges {
126127
}
127128
}
128129
else {
129-
return workDayStarted ? workDayEnded ? 'green-background' : 'red-background' : plannedStarted ? 'grey-background' : message ? 'grey-background' : 'white-background';
130+
return workDayStarted ? workDayEnded ? 'green-background' : 'red-background' : plannedStarted ? 'grey-background' : message || workerComment ? 'grey-background' : 'white-background';
130131
}
131132
}
132133
catch (e) {
@@ -137,9 +138,12 @@ export class TimePlanningsTableComponent implements OnInit, OnChanges {
137138

138139
getCellTextColor(row: any, field: string): string {
139140
const planHours = row.planningPrDayModels[field]?.planHours;
140-
const workDayStarted = row.planningPrDayModels[field]?.workDayStarted;
141+
const plannedStarted = row.planningPrDayModels[field]?.plannedStartOfShift1
142+
const workDayStarted = row.planningPrDayModels[field]?.start1StartedAt || row.planningPrDayModels[field]?.start2StartedAt;
141143
const workDayEnded = row.planningPrDayModels[field]?.workDayEnded;
142144
const isInOlderThanToday = new Date(row.planningPrDayModels[field]?.date) < new Date();
145+
const message = row.planningPrDayModels[field]?.message;
146+
const workerComment = row.planningPrDayModels[field]?.workerComment;
143147
if (planHours > 0) {
144148
if (workDayStarted) {
145149
//console.log('getCellTextColor', row, field, planHours, workDayStarted, workDayEnded);
@@ -149,14 +153,15 @@ export class TimePlanningsTableComponent implements OnInit, OnChanges {
149153
return isInOlderThanToday ? 'red-text' : 'black-text';
150154
}
151155
} else {
152-
if (workDayStarted) {
153-
return 'black-text';
154-
}
155-
else {
156-
return isInOlderThanToday ? 'red-text' : 'black-text';
157-
}
156+
return workDayStarted ? workDayEnded ? 'black-text' : 'red-text' : plannedStarted ? 'white-text' : message || workerComment ? 'black-text' : 'white-text';
157+
// if (workDayStarted) {
158+
// return 'black-text';
159+
// }
160+
// else {
161+
// return isInOlderThanToday ? 'red-text' : 'black-text';
162+
// }
158163
}
159-
return 'black-text';
164+
// return 'black-text';
160165
}
161166

162167
getCellTextColorForDay(row: any, field: string): string {

0 commit comments

Comments
 (0)