Sort files in logical rather than dictionary order (#9012)

contributors.md

@@ -121,6 +121,10 @@ The following people are not part of the development team, but have been contrib
 * Jason Myre (jmyre1999)
 * Nicole Wright (nicolewright)
 * Josh Tucker (joshtucker132)
+* Hussein Okasha (Hokasha2016)
+* Brandon Dupree (Bdupree5)
+* Zetao Ye (ZbrettonYe)
+* Jordan Arevalos (Jarevalos2017)
 
 ## Toolchain
 * (Balletie) - macOS

distribution/changelog.txt

@@ -5,6 +5,7 @@
 - Feature: [#8481] Multi-threaded rendering.
 - Feature: [#8919] Allow setting ride price from console.
 - Feature: [#8963] Add missing Czech letters to sprite font, use sprite font for Czech.
+- Change: [#7877] Files are now sorted in logical rather than dictionary order.
 - Change: [#8688] Move common actions from debug menu into cheats menu.
 - Fix: [#5579] Network desync immediately after connecting.
 - Fix: [#5905] Urban Park merry-go-round has entrance and exit swapped (original bug).

src/openrct2-ui/windows/LoadSave.cpp

@@ -786,15 +786,15 @@ static bool list_item_sort(LoadSaveListItem& a, LoadSaveListItem& b)
     switch (gConfigGeneral.load_save_sort)
     {
         case SORT_NAME_ASCENDING:
-            return strcicmp(a.name.c_str(), b.name.c_str()) < 0;
+            return strlogicalcmp(a.name.c_str(), b.name.c_str()) < 0;
         case SORT_NAME_DESCENDING:
-            return -strcicmp(a.name.c_str(), b.name.c_str()) < 0;
+            return -strlogicalcmp(a.name.c_str(), b.name.c_str()) < 0;
         case SORT_DATE_DESCENDING:
             return -difftime(a.date_modified, b.date_modified) < 0;
         case SORT_DATE_ASCENDING:
             return difftime(a.date_modified, b.date_modified) < 0;
         default:
-            return strcicmp(a.name.c_str(), b.name.c_str()) < 0;
+            return strlogicalcmp(a.name.c_str(), b.name.c_str()) < 0;
     }
 }