From 53e04551e652de1846baeb86f765ae488743f5bb Mon Sep 17 00:00:00 2001
From: ccremer <github.account@chrigel.net>
Date: Mon, 16 Jan 2023 14:14:29 +0100
Subject: [PATCH] Align and colorize columns in the employee report

---
 pkg/web/employeereport/employeereport_view.go |  3 ++
 templates/employeereport.html                 | 44 +++++++++++++------
 2 files changed, 33 insertions(+), 14 deletions(-)

diff --git a/pkg/web/employeereport/employeereport_view.go b/pkg/web/employeereport/employeereport_view.go
index 6fdbd86..ab206ed 100644
--- a/pkg/web/employeereport/employeereport_view.go
+++ b/pkg/web/employeereport/employeereport_view.go
@@ -64,9 +64,12 @@ func (v *reportView) getValuesForReport(report timesheet.Report, previousPayslip
 		"WorkedHours":                     v.FormatDurationInHours(report.Summary.TotalWorkedTime),
 		"OutOfOfficeHours":                v.FormatDurationInHours(report.Summary.TotalOutOfOfficeTime),
 		"OvertimeHours":                   v.FormatDurationInHours(report.Summary.TotalOvertime),
+		"OvertimeClassName":               v.OvertimeClassname(report.Summary.TotalOvertime),
 		"PreviousBalance":                 previousBalanceCellText,
 		"NextBalance":                     nextBalanceCellText,
+		"NextBalanceClassName":            v.OvertimeClassname(nextBalance),
 		"ProposedBalance":                 proposedBalanceCellText,
+		"ProposedBalanceClassName":        v.OvertimeClassname(proposedBalance),
 		"OvertimeBalanceEditEnabled":      nextPayslip != nil,
 		"OvertimeBalanceEditPreviewValue": overtimeBalanceEditPreview,
 		"ValidationError":                 validationErrorList.Error(),
diff --git a/templates/employeereport.html b/templates/employeereport.html
index ac40b71..9282648 100644
--- a/templates/employeereport.html
+++ b/templates/employeereport.html
@@ -50,6 +50,22 @@
         top: 0;
         z-index: 1;
     }
+
+    .Overtime {
+        color: #005AB5;
+    }
+
+    .Undertime {
+        color: #DC3220;
+    }
+    @media (min-width: 2000px) {
+        .container {
+            max-width: 2000px;
+        }
+        .table {
+            table-layout: fixed;
+        }
+    }
 </style>
 <h1>Attendances for {{ .Month }} {{ .Year }}</h1>
 <div id="alerts">
@@ -73,13 +89,13 @@ <h1>Attendances for {{ .Month }} {{ .Year }}</h1>
     <tr class="table-secondary">
         <th scope="col">Name</th>
         <th scope="col">Leaves</th>
-        <th scope="col">Excused hours</th>
-        <th scope="col">Worked hours</th>
-        <th scope="col">(Out of office hours, real)</th>
-        <th scope="col">{{ .LastMonth }} Payslip</th>
-        <th scope="col">Overtime delta</th>
-        <th scope="col">Proposed balance</th>
-        <th scope="col">{{ .Month }} Payslip</th>
+        <th scope="col" class="text-end">Excused hours</th>
+        <th scope="col" class="text-end">Worked hours</th>
+        <th scope="col" class="text-end">(Out of office hours, real)</th>
+        <th scope="col" class="text-end">{{ .LastMonth }} Payslip</th>
+        <th scope="col" class="text-end">Overtime delta</th>
+        <th scope="col" class="text-end">Proposed balance</th>
+        <th scope="col" class="text-end">{{ .Month }} Payslip</th>
         <th scope="col">Save in {{ .Month }} Payslip</th>
     </tr>
     </thead>
@@ -91,13 +107,13 @@ <h1>Attendances for {{ .Month }} {{ .Year }}</h1>
             {{- with .ValidationError }}<br>⚠️ {{ . }}{{ end -}}
         </td>
         <td>{{ .Leaves }}d</td>
-        <td>{{ .ExcusedHours }}</td>
-        <td>{{ .WorkedHours }}</td>
-        <td>{{ .OutOfOfficeHours }}</td>
-        <td>{{ .PreviousBalance }}</td>
-        <td>{{ .OvertimeHours }}</td>
-        <td>{{ .ProposedBalance }}</td>
-        <td id="td-nextbalance-{{ .EmployeeID }}">{{ .NextBalance }}</td>
+        <td class="text-end font-monospace">{{ .ExcusedHours }}</td>
+        <td class="text-end font-monospace">{{ .WorkedHours }}</td>
+        <td class="text-end font-monospace">{{ .OutOfOfficeHours }}</td>
+        <td class="text-end font-monospace">{{ .PreviousBalance }}</td>
+        <td class="text-end font-monospace {{ .OvertimeClassName }}">{{ .OvertimeHours }}</td>
+        <td class="text-end font-monospace {{ .ProposedBalanceClassName }}">{{ .ProposedBalance }}</td>
+        <td class="text-end font-monospace {{ .NextBalanceClassName }}" id="td-nextbalance-{{ .EmployeeID }}">{{ .NextBalance }}</td>
         <td>
             {{- if .OvertimeBalanceEditEnabled }}
             <div class="mb-3">