fix: colorize negative scores in tables

Author: FreekBes

src/handlers/filters.ts

@@ -18,6 +18,15 @@ export const setupNunjucksFilters = function(app: Express): void {
 		return num.toString().replace(/\./, ',').replace(/\B(?=(\d{3})+(?!\d))/g, '.');
 	});
 
+	// Add formatting for numbers to be colored red
+	nunjucksEnv.addFilter('colorNegative', (num: number | string) => {
+		const number = typeof num === 'string' ? parseFloat(num.replace(/\./g, '').replace(',', '.')) : num;
+		if (number < 0) {
+			return `<span style="color: orangered; font-weight: bold;">${num}</span>`;
+		}
+		return num;
+	});
+
 	// Add formatting filter for seconds to hh:mm format
 	nunjucksEnv.addFilter('formatSeconds', (seconds: number) => {
 		const hours = Math.floor(seconds / 3600);

templates/admin/points/history.njk

@@ -28,7 +28,7 @@
 				<td><a href="/admin/points/history/{{ score.id }}" target="scorejson">{{ score.id }}</a></td>
 				<td title="{{ score.created_at }}">{{ score.created_at | timeAgo }}</td>
 				<td>{{ coalitions[score.coalition_id].intra_coalition.name | striptags(true) | escape }}</td>
-				<td>{{ score.amount }}</td>
+				<td>{{ score.amount | thousands | colorNegative }}</td>
 				<td><a href="/profile/{{ score.user.intra_user.login | striptags(true) | escape }}">{{ score.user.intra_user.login | striptags(true) | escape }}</a></td>
 				<td style="white-space: normal; word-wrap: break-word;">{{ score.reason | striptags(true) | escape }}</td>
 				<td>{{ score.fixed_type_id if score.fixed_type_id else "" }}</td>

templates/admin/points/manual/added.njk

@@ -42,7 +42,7 @@
 			<tr class="points-added-row" style="background: {{ score.coalition.intra_coalition.color | rgba(0.25) }}">
 				<td><a href="/profile/{{ score.user.intra_user.login }}">{{ score.user.intra_user.login }}</a></td>
 				<td>{{ score.coalition.intra_coalition.name | striptags(true) | escape }}</td>
-				<td>{{ score.amount | thousands }}</td>
+				<td>{{ score.amount | thousands | colorNegative }}</td>
 				<td>{{ score.reason | striptags(true) | escape }}</td>
 			</tr>
 		{% endfor %}

templates/admin/points/manual/custom.njk

@@ -139,7 +139,7 @@ else {
 					<td>{{ score.created_at | timeAgo }}</td>
 					<td><a href="/profile/{{ score.user.intra_user.login }}/">{{ score.user.intra_user.login }}</a></td>
 					<td>{{ score.coalition.intra_coalition.name | striptags(true) | escape }}</td>
-					<td>{{ score.amount | thousands }}</td>
+					<td>{{ score.amount | thousands | colorNegative }}</td>
 					<td>{{ score.reason | striptags(true) | escape }}</td>
 				</tr>
 			{% endfor %}

templates/coalition.njk

@@ -120,7 +120,7 @@
 								<tr>
 									<td title="{{ score.created_at }}">{{ score.created_at | timeAgo }}</td>
 									<td><a href="/profile/{{ score.user.intra_user.login | striptags(true) | escape }}">{{ score.user.intra_user.login | striptags(true) | escape }}</a></td>
-									<td>{{ score.amount }}</td>
+									<td>{{ score.amount | thousands | colorNegative}}</td>
 									<td style="white-space: normal; word-wrap: break-word;"><small>{{ score.reason | striptags(true) | escape }}</small></td>
 								</tr>
 							{% endfor %}
@@ -152,7 +152,7 @@
 								<tr>
 									<td title="{{ score.created_at }}">{{ score.created_at | timeAgo }}</td>
 									<td><a href="/profile/{{ score.user.intra_user.login | striptags(true) | escape }}">{{ score.user.intra_user.login | striptags(true) | escape }}</a></td>
-									<td>{{ score.amount }}</td>
+									<td>{{ score.amount | thousands | colorNegative }}</td>
 									<td style="white-space: normal; word-wrap: break-word;"><small>{{ score.reason | striptags(true) | escape }}</small></td>
 								</tr>
 							{% endfor %}

templates/profile.njk

@@ -76,7 +76,7 @@
 							{% for score in latestScores %}
 								<tr style="background: {{ score.coalition.intra_coalition.color | rgba(0.25) }}">
 									<td title="{{ score.created_at }}">{{ score.created_at | timeAgo }}</td>
-									<td>{{ score.amount | thousands }}</td>
+									<td>{{ score.amount | thousands | colorNegative }}</td>
 									<td style="white-space: normal; word-wrap: break-word;">{{ score.reason | striptags(true) | escape }}</td>
 								</tr>
 							{% endfor %}