Automatic merge of T1.5.1-1088-g264349b1ce and 21 pull requests

- Pull request #891 at 9a1d6b2: Auto save
- Pull request #959 at 2452cb0: Fix TrackViewer crash on big zoom value
- Pull request #972 at e90a2aa: On Map window color changed switch or signal is not changed
- Pull request #839 at d00beb9: First phase of https://blueprints.launchpad.net/or/+spec/additional-cruise-control-parameters
- Pull request #882 at 8f695a4: Blueprint/train car operations UI window
- Pull request #892 at 1f5ba4c: Signal Function OPP_SIG_ID_TRAINPATH
- Pull request #922 at a3bc9e7: Autopilot for timetable mode
- Pull request #953 at a519452: Fix Lights Crash on Corrupt Shapes
- Pull request #962 at 46d0472: Fix pantographs on unpowered cars
- Pull request #970 at 6fa5eed: feat: Remove unnecessary Windows.Forms usage
- Pull request #973 at c35be87: fix: Using singular target framework to allow for different platforms across different projects
- Pull request #900 at c27f32d: DMI updates
- Pull request #903 at 9dabe97: Downloading route content (Github, zip)
- Pull request #799 at dfc715e: Consolidated wind simulation
- Pull request #876 at f92de76: docs: add source for documents previously on website to source Documentation folder
- Pull request #885 at 067662e: feat: Add notifications to Menu
- Pull request #896 at 5866028: First implementation of https://blueprints.launchpad.net/or/+spec/specific-sounds-for-ai-trains
- Pull request #946 at 91a03af: Advanced track sounds
- Pull request #952 at 8347095: Investigation - Pulsing graphics
- Pull request #954 at f837ffd: Multiple Track Profiles & Superelevation Improvements
- Pull request #968 at a88cb26: Initial build of adding track section identifier for rack railway

Author: openrails-bot

Source/Menu/MainForm.Designer.cs

@@ -326,6 +326,7 @@ void CheckForUpdate()
             }, _ =>
             {
                 NotificationManager.CheckNotifications();
+                UpdateNotificationPageAlert();
             });
         }
 
@@ -1594,7 +1595,7 @@ private void ToggleNotificationPages()
             {
                 NotificationManager.ArePagesVisible = true; // Set before calling ShowNotifcations()
                 ShowNotificationPages();
-                FiddleNewNotificationPageCount();
+                UpdateNotificationPageAlert();
             }
             else
             {
@@ -1603,12 +1604,6 @@ private void ToggleNotificationPages()
             }
         }
 
-        private void FiddleNewNotificationPageCount()
-        {
-            //NotificationManager.LastPageViewed = 1; //TODO
-            UpdateNotificationPageAlert();
-        }
-
         public void ShowNotificationPages()
         {
             Win32.LockWindowUpdate(Handle);
@@ -1621,7 +1616,13 @@ public void ShowNotificationPages()
 
         public void UpdateNotificationPageAlert()
         {
-            if (NotificationManager.LastPageViewed >= NotificationManager.NewPageCount)
+            if (NotificationManager.NewPagesViewed < NotificationManager.NewPageCount)
+            {
+                pbNotificationsSome.Visible = true;
+                lblNotificationCount.Visible = true;
+                lblNotificationCount.Text = $"{NotificationManager.NewPageCount - NotificationManager.NewPagesViewed}";
+            }
+            else
             {
                 pbNotificationsSome.Visible = false;
                 lblNotificationCount.Visible = false;

Source/Menu/MainForm.cs

@@ -30,6 +30,7 @@
 using System.Drawing;
 using System.Windows.Forms;
 using System.Linq;
+using System.Diagnostics;
 
 //TODO indicate number of never read notifications
 
@@ -45,11 +46,12 @@ namespace ORTS
     public class NotificationManager
     {
         public bool ArePagesVisible = false;
-        // New notifications are those with a date after the NotificationsReadDate.
+
         // Notifications are listed in reverse date order, with the newest one at the front.
-        // We don't track the reading of each notification but set the NewNotificationCount = 0 after the last of the new ones has been read.
-        public int NewPageCount = 1;
-        public int LastPageViewed = 0;
+        // We track the reading of each new notification but set the NewPageCount = 0 after the last of the new ones has been read.
+        public string LastViewDate; // This is the date when notifications were last viewed and is saved in yyyy-MM-dd format as a hidden UserSetting.
+        public int NewPageCount;    // New notifications are those with a date after the LastViewDate.
+        public int NewPagesViewed;  // Viewing always starts at the first, newest notification. The user sees (NewPageCount - NewpagesViewed) 
 
         public Notifications Notifications; // An object defined by the JSON schema
         public int CurrentNotificationNo = 0;
@@ -140,6 +142,11 @@ public Notifications GetNotifications()
             var jsonSettings = new JsonSerializerSettings { TypeNameHandling = TypeNameHandling.Auto };
             var jsonInput = JsonConvert.DeserializeObject<Notifications>(notificationsSerial, jsonSettings);
 
+            NewPageCount = 0;
+            NewPagesViewed = 0;
+            LastViewDate = Settings.LastViewNotificationDate;
+            if (LastViewDate == "") LastViewDate = "2024-01-01"; // Date of this code - i.e. before Notifications went public
+
             //TODO Cache jsonInput
 
             return jsonInput;
@@ -164,12 +171,19 @@ private string GetRemoteJson()
 
         private List<Notification> IncludeValid(List<Notification> list)
         {
+            NewPageCount = 0;
+
             var filteredList = new List<Notification>();
             foreach (var n in list)
             {
                 if (AreNotificationChecksMet(n))
                 {
-                    if (n.Date == "none") n.Date = " none"; // UpdateChannel = "none" found; push this to end of the list
+                    if (n.Date == "none") 
+                        n.Date = " none"; // UpdateChannel = "none" found; push this to end of the list
+                    else
+                    {
+                        if (String.Compare(LastViewDate, n.Date) == -1) NewPageCount++;
+                    }
                     filteredList.Insert(0, n); // Add to head of list to provide a basic pre-sort.
                 }
             }
@@ -186,6 +200,8 @@ private List<Notification> SortByDate(List<Notification> list)
         /// </summary>
         public void PopulatePage()
         {
+            Settings.LastViewNotificationDate = $"{DateTime.Today:yyyy-MM-dd}";    
+            Settings.Save("LastViewNotificationDate");  // Saves the date on any viewing of notifications
             Page = new NotificationPage(MainForm, Panel, this); 
 
             if (UpdateManager.LastCheckError != null || Error != null)
@@ -209,7 +225,6 @@ public void PopulatePage()
                 Page.NDetailList.Add(new NRetryControl(Page, "Retry", 140, "Try again to fetch notifications", MainForm));
             }
 
-            NewPageCount = 1;
             var list = Notifications.NotificationList;
             var n = list[CurrentNotificationNo];
             LogNotification(n);
@@ -590,7 +605,12 @@ public OverrideParameterList GetOverrideParameters()
         public void ChangePage(int step)
         {
             CurrentNotificationNo += step;
-            //SetVisibility(step);
+            if (step > 0
+                && CurrentNotificationNo > NewPagesViewed   // and this is a new unviewed page
+                && CurrentNotificationNo <= NewPageCount) // and there are still new unviewed pages to be viewed
+            {
+                NewPagesViewed++;
+            }
             MainForm.ShowNotificationPages();
         }
 

Source/Menu/Notifications/NotificationManager.cs

@@ -210,7 +210,6 @@ public enum DirectXFeature
         public int AntiAliasing { get; set; }
 
         // Simulation settings:
-
         [Default(false)]
         public bool SimpleControlPhysics { get; set; }
         [Default(true)]
@@ -259,7 +258,6 @@ public enum DirectXFeature
         [Default(false)]
         public bool DataLogStationStops { get; set; }
 
-
         // Timetable settings:
         [Default(true)]
         public bool TTUseRestartDelays { get; set; }
@@ -416,6 +414,8 @@ public string DirectXFeatureLevel
         public int[] WindowPosition_ComposeMessage { get; set; }
         [Default(new[] { 100, 0 })]
         public int[] WindowPosition_TrainList { get; set; }
+        [Default("")]
+        public string LastViewNotificationDate { get; set; }
 
         // Menu-game communication settings:
         [Default(false)]