From bd0ba2a120b7a5067c876ce8a5fbd4539132819f Mon Sep 17 00:00:00 2001
From: Birte Kristina Friesel <derf@finalrewind.org>
Date: Wed, 10 Apr 2024 18:44:07 +0200
Subject: train details: indicate on time / delayed time stamps by color

also: revert back to showing scheduled time first and real/estimated time next
---
 sass/app.scss                    | 29 +++++++++++++++++++++++++++--
 sass/dark.scss                   |  3 +++
 sass/light.scss                  |  3 +++
 templates/_train_details.html.ep | 37 ++++++++++++++++++++++---------------
 4 files changed, 55 insertions(+), 17 deletions(-)

diff --git a/sass/app.scss b/sass/app.scss
index 767dcad..f24ac9c 100644
--- a/sass/app.scss
+++ b/sass/app.scss
@@ -618,17 +618,42 @@ div.app {
 				list-style-type: circle;
 			}
 
+			.time-early {
+				color: $early-stop-color;
+			}
+
+			.time-delayed {
+				color: $delayed-stop-color;
+			}
+
+			.time-sched-only {
+				color: $delayed-stop-color;
+			}
+
+			.time-sched-ontime {
+				color: $early-stop-color;
+			}
+
+
 			.annotation {
 				color: $fg2;
 				list-style-type: none;
 				padding-left: 3em;
 			}
 
-			.sched-delayed:before {
+			.-sched:before {
+				content: " ";
+			}
+
+			.time-sched:after {
+				content: " ";
+			}
+
+			.time-sched-only:before {
 				content: "(";
 			}
 
-			.sched-delayed:after {
+			.time-sched-only:after {
 				content: ")";
 			}
 
diff --git a/sass/dark.scss b/sass/dark.scss
index 83cb520..c0e8d2b 100644
--- a/sass/dark.scss
+++ b/sass/dark.scss
@@ -39,6 +39,9 @@ $undelaynorm-color: #99dd99;
 $additional-stop-color: #77ff77;
 $cancelled-stop-color: #ff7777;
 
+$early-stop-color: #ccffcc;
+$delayed-stop-color: #ff9999;
+
 $cancelled-bg-color: #512f00;
 $past-bg-color: $bg05;
 
diff --git a/sass/light.scss b/sass/light.scss
index d8f53cb..1dee6a9 100644
--- a/sass/light.scss
+++ b/sass/light.scss
@@ -39,6 +39,9 @@ $undelaynorm-color: #338833;
 $additional-stop-color: #009900;
 $cancelled-stop-color: #cc0000;
 
+$early-stop-color: #007700;
+$delayed-stop-color: #990000;
+
 $cancelled-bg-color: #ffe7d0;
 $past-bg-color: $bg05;
 
diff --git a/templates/_train_details.html.ep b/templates/_train_details.html.ep
index 2f89cde..709a1ac 100644
--- a/templates/_train_details.html.ep
+++ b/templates/_train_details.html.ep
@@ -271,13 +271,13 @@
             generic-stop
 %         }
 %         if (($stop->{rt_dep} and $stop->{dep_delay}) or (not $stop->{rt_dep} and $stop->{rt_arr} and $stop->{arr_delay})) {
-            "><%= ($stop->{rt_dep} // $stop->{rt_arr})->strftime('%H:%M') %> <span class="sched-delayed"><%= ($stop->{sched_dep} // $stop->{sched_arr})->strftime('%H:%M') %></span>
+            "><span class="time-sched-only"><%= ($stop->{sched_dep} // $stop->{sched_arr})->strftime('%H:%M') %></span> <span class="time-delayed"><%= ($stop->{rt_dep} // $stop->{rt_arr})->strftime('%H:%M') %></span>
+%         }
+%         elsif (($stop->{rt_dep} and defined $stop->{dep_delay}) or (not $stop->{rt_dep} and $stop->{rt_arr} and defined $stop->{arr_delay})) {
+            "><span class="time-sched-ontime"><%= ($stop->{sched_dep} // $stop->{sched_arr}) ? ($stop->{sched_dep} // $stop->{sched_arr})->strftime('%H:%M') : q{} %></span>
 %         }
 %         else {
-            "><%= ($stop->{sched_dep} // $stop->{sched_arr}) ? ($stop->{sched_dep} // $stop->{sched_arr})->strftime('%H:%M') : q{} %>
-%           if ($stop->{rt_bogus}) {
-              <i class="material-icons" aria-label="Echtzeitdaten fehlen">gps_off</i>
-%           }
+            "><span class="time-sched"><%= ($stop->{sched_dep} // $stop->{sched_arr}) ? ($stop->{sched_dep} // $stop->{sched_arr})->strftime('%H:%M') : q{} %></span>
 %         }
 %         if ($stop->{tz_offset} and $stop->{local_dt_da}) {
             (lokal <%= $stop->{local_dt_da}->strftime('%H:%M') %>)
@@ -305,16 +305,23 @@
             </li>
 %         }
           <li class="<%= $departure->{is_cancelled} ? 'cancelled-stop' : q{} %> <%= $departure->{isPast} ? 'past-stop' : 'future-stop' %>">
-%=          $departure->{departure} // $departure->{arrival} // $departure->{sched_departure} // $departure->{sched_arrival} // q{}
 %           if ($departure->{departure} and $departure->{sched_departure} and $departure->{departure} ne $departure->{sched_departure}) {
-              <span class="sched-delayed"><%= $departure->{sched_departure} // $departure->{sched_arrival} // q{} %></span>
+              <span class="time-sched-only"><%= $departure->{sched_departure} // $departure->{sched_arrival} // q{} %></span><span class="time-delayed">
+%           }
+%           elsif ($departure->{departure} and $departure->{sched_departure} and $departure->{departure} eq $departure->{sched_departure} and not $departure->{no_realtime_yet}) {
+              <span class="time-sched-ontime">
 %           }
 %           elsif ($departure->{arrival} and $departure->{sched_arrival} and $departure->{arrival} ne $departure->{sched_arrival}) {
-              <span class="sched-delayed"><%= $departure->{sched_departure} // $departure->{sched_arrival} // q{} %></span>
+              <span class="time-sched-only"><%= $departure->{sched_departure} // $departure->{sched_arrival} // q{} %></span><span class="time-delayed">
 %           }
-%           if ($departure->{missing_realtime} or $departure->{no_realtime_yet}) {
-              <i class="material-icons" aria-label="Echtzeitdaten fehlen">gps_off</i>
+%           elsif ($departure->{arrival} and $departure->{sched_arrival} and $departure->{arrival} eq $departure->{sched_arrival} and not $departure->{no_realtime_yet}) {
+              <span class="time-sched-ontime">
 %           }
+%           else {
+              <span class="time-sched">
+%           }
+%=          $departure->{departure} // $departure->{arrival} // $departure->{sched_departure} // $departure->{sched_arrival} // q{}
+            </span>
 %           if ($departure->{tz_offset} and $departure->{local_dt_da}) {
               (lokal <%= $departure->{local_dt_da}->strftime('%H:%M') %>)
 %           }
@@ -355,13 +362,13 @@
             generic-stop
 %         }
 %         if (($stop->{rt_arr} and $stop->{arr_delay}) or (not $stop->{rt_arr} and $stop->{rt_dep} and $stop->{dep_delay})) {
-            "><%= ($stop->{rt_arr} // $stop->{rt_dep})->strftime('%H:%M') %> <span class="sched-delayed"><%= ($stop->{sched_arr} // $stop->{sched_dep})->strftime('%H:%M') %></span>
+            "><span class="time-sched-only"><%= ($stop->{sched_arr} // $stop->{sched_dep})->strftime('%H:%M') %></span> <span class="time-delayed"><%= ($stop->{rt_arr} // $stop->{rt_dep})->strftime('%H:%M') %></span>
+%         }
+%         elsif (($stop->{rt_arr} and defined $stop->{arr_delay}) or (not $stop->{rt_arr} and $stop->{rt_dep} and defined $stop->{dep_delay})) {
+            "><span class="time-sched-ontime"><%= ($stop->{sched_arr} // $stop->{sched_dep}) ? ($stop->{sched_arr} // $stop->{sched_dep})->strftime('%H:%M') : q{} %></span>
 %         }
 %         else {
-            "><%= ($stop->{sched_arr} // $stop->{sched_dep}) ? ($stop->{sched_arr} // $stop->{sched_dep})->strftime('%H:%M') : q{} %>
-%           if ($stop->{rt_bogus}) {
-              <i class="material-icons" aria-label="Echtzeitdaten fehlen">gps_off</i>
-%           }
+            "><span class="time-sched"><%= ($stop->{sched_arr} // $stop->{sched_dep}) ? ($stop->{sched_arr} // $stop->{sched_dep})->strftime('%H:%M') : q{} %></span>
 %         }
 %         if ($stop->{tz_offset} and $stop->{local_dt_ad}) {
             (lokal <%= $stop->{local_dt_ad}->strftime('%H:%M') %>)
-- 
cgit v1.2.3