refactor(DebugLogViewerSettings): change port setting type from string to int and update related logic

- Modified the DebugLogViewerPort setting to use an integer type instead of a string, allowing for better validation and range enforcement.
- Updated the UI bindings and localization files to reflect the change in port type, removing the placeholder text and ensuring clarity in descriptions.
- Simplified the port value handling by directly clamping the integer value, enhancing the overall robustness of the settings management.

Author: BAKAOLC

Settings/Integration/RitsuLibModSettingsBootstrap.DebugLogViewerPage.cs

@@ -21,15 +21,16 @@ private static void RegisterDebugLogViewerPage(RitsuLibModSettingsUiBindings ui)
                             ui.DebugLogViewerAutoOpen,
                             T("ritsulib.debugLogViewer.autoOpen.description",
                                 "When enabled, RitsuLib waits 3 seconds after starting the viewer. If no browser page is listening, it opens one."))
-                        .AddString(
+                        .AddIntSlider(
                             "debug_log_viewer_port",
                             T("ritsulib.debugLogViewer.port.label", "Viewer port"),
                             ui.DebugLogViewerPort,
-                            T("ritsulib.debugLogViewer.port.placeholder", "18742"),
-                            5,
+                            1,
+                            65535,
+                            1,
+                            value => value.ToString(),
                             T("ritsulib.debugLogViewer.port.description",
-                                "Loopback HTTP port. If the port is busy, RitsuLib tries the following ports in order. Changes apply on next launch."),
-                            value => RitsuLibModSettingsUiBindings.TryParsePort(value, out _))
+                                "Loopback HTTP port. If the port is busy, RitsuLib tries the following ports in order. Changes apply on next launch."))
                         .AddButton(
                             "debug_log_viewer_open_now",
                             T("ritsulib.debugLogViewer.openNow.label", "Open viewer"),

Settings/Integration/RitsuLibModSettingsUiBindings.cs

@@ -15,7 +15,7 @@ private RitsuLibModSettingsUiBindings()
         public IModSettingsValueBinding<bool> DebugCompatLocTable { get; private init; } = null!;
         public IModSettingsValueBinding<bool> DebugCompatUnlockEpoch { get; private init; } = null!;
         public IModSettingsValueBinding<bool> DebugLogViewerAutoOpen { get; private init; } = null!;
-        public IModSettingsValueBinding<string> DebugLogViewerPort { get; private init; } = null!;
+        public IModSettingsValueBinding<int> DebugLogViewerPort { get; private init; } = null!;
 
         public IModSettingsValueBinding<bool> DebugCompatAncientArchitect { get; private init; } =
             null!;
@@ -153,16 +153,12 @@ public static RitsuLibModSettingsUiBindings Create()
                         (settings, value) => settings.DebugLogViewerAutoOpen = value),
                     () => defaults.DebugLogViewerAutoOpen),
                 DebugLogViewerPort = ModSettingsBindings.WithDefault(
-                    ModSettingsBindings.Global<RitsuLibSettings, string>(
+                    ModSettingsBindings.Global<RitsuLibSettings, int>(
                         Const.ModId,
                         Const.SettingsKey,
-                        settings => Math.Clamp(settings.DebugLogViewerPort, 1, 65535).ToString(),
-                        (settings, value) =>
-                        {
-                            if (TryParsePort(value, out var port))
-                                settings.DebugLogViewerPort = port;
-                        }),
-                    () => defaults.DebugLogViewerPort.ToString()),
+                        settings => Math.Clamp(settings.DebugLogViewerPort, 1, 65535),
+                        (settings, value) => settings.DebugLogViewerPort = Math.Clamp(value, 1, 65535)),
+                    () => defaults.DebugLogViewerPort),
                 ModSourceHoverTipsEnabled = ModSettingsBindings.WithDefault(
                     ModSettingsBindings.Global<RitsuLibSettings, bool>(
                         Const.ModId,
@@ -515,9 +511,5 @@ private static string NormalizeToastAnimation(string? value)
             };
         }
 
-        internal static bool TryParsePort(string? value, out int port)
-        {
-            return int.TryParse(value?.Trim(), out port) && port is >= 1 and <= 65535;
-        }
     }
 }

Settings/Localization/ModSettingsUi/eng.json

@@ -220,7 +220,6 @@
   "ritsulib.debugLogViewer.autoOpen.label": "Open browser automatically",
   "ritsulib.debugLogViewer.autoOpen.description": "When enabled, RitsuLib waits 3 seconds after starting the viewer. If no browser page is listening, it opens one.",
   "ritsulib.debugLogViewer.port.label": "Viewer port",
-  "ritsulib.debugLogViewer.port.placeholder": "18742",
   "ritsulib.debugLogViewer.port.description": "Loopback HTTP port. If the port is busy, RitsuLib tries the following ports in order. Changes apply on next launch.",
   "ritsulib.debugLogViewer.openNow.label": "Open viewer",
   "ritsulib.debugLogViewer.openNow.button": "Open log viewer",

Settings/Localization/ModSettingsUi/zhs.json

@@ -220,7 +220,6 @@
   "ritsulib.debugLogViewer.autoOpen.label": "自动打开浏览器",
   "ritsulib.debugLogViewer.autoOpen.description": "启用后，RitsuLib 会在查看器启动后等待 3 秒；如果没有浏览器页面正在监听，就自动打开一个页面。",
   "ritsulib.debugLogViewer.port.label": "查看器端口",
-  "ritsulib.debugLogViewer.port.placeholder": "18742",
   "ritsulib.debugLogViewer.port.description": "本机 HTTP 端口。如果端口被占用，RitsuLib 会按顺序尝试后续端口。修改会在下次启动时生效。",
   "ritsulib.debugLogViewer.openNow.label": "打开查看器",
   "ritsulib.debugLogViewer.openNow.button": "打开日志查看器",