Renamed resolution-related commands to be easier to understand the purpose.

Author: DavidF-Dev

CHANGELOG.md

@@ -8,6 +8,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - Added LogException methods.
 - Changed the size of the input field so that the text is larger.
 - Changed enum command parameter to support namespaces to distinguish between enums with same names.
+- Changed resolution command names to be easier to understand.
 - Fixed strings being displayed incorrectly by cs_evaluate.
 - Fixed issue with cached enum types.
 - Fixed input field not unfocusing when the developer console window is closed.

DOCUMENTATION.md

@@ -102,7 +102,8 @@ Listed below are all the built-in commands that come with the developer console
 - ``fullscreen (Boolean)enabled``: Query or set whether the window is full screen.
 - ``fullscreen_mode (FullScreenMode)mode``: Query or set the full screen mode.
 - ``vsync (Int32)vSyncCount``: Query or set whether VSync is enabled.
-- ``resolution``: Display the current screen resolution.
+- ``monitor_size``: Display the current monitor resolution.
+- ``window_size``: Display the current window resolution.
 - ``targetfps (Int32)targetFrameRate``: Query or set the target frame rate.
 
 ### Camera commands

Runtime/DevConsoleMono.cs

@@ -1691,15 +1691,8 @@ private void InitBuiltInCommands()
                         return;
                     }
 
-                    string result = "";
-                    foreach (InputKey key in _bindings.Keys)
-                    {
-                        result += $"<i>{key}</i>: \"{_bindings[key]}\"\n";
-                    }
-                    result = result.Remove(result.Length - 2);
-
                     LogSeperator($"Key bindings ({_bindings.Count})");
-                    Log(result);
+                    Log(string.Join("\n", _bindings.Keys.Select(x => $"<i>{x}</i>: \"{_bindings[x]}\"")));
                     LogSeperator();
                 }
             ));
@@ -1849,7 +1842,7 @@ private void InitBuiltInCommands()
                 "vsync",
                 "",
                 "Query or set whether VSync is enabled",
-                Parameter.Create("vSyncCount", "The number of VSyncs that should pass between each frame (0, 1, 2, 3, or 4)."),
+                Parameter.Create("vSyncCount", "The number of VSyncs that should pass between each frame (0, 1, 2, 3, or 4)"),
                 i =>
                 {
                     if (i < 0 || i > 4)
@@ -1865,17 +1858,17 @@ private void InitBuiltInCommands()
             ));
 
             AddCommand(Command.Create(
-                "resolution",
-                "",
+                "monitor_size",
+                "monitor_resolution",
                 "Display the current monitor resolution",
-                () => LogVariable("Resolution", Screen.currentResolution)
+                () => LogVariable("Monitor size", Screen.currentResolution)
             ));
 
             AddCommand(Command.Create(
-                "screen_size",
-                "",
+                "window_size",
+                "window_resolution",
                 "Display the current window resolution",
-                () => LogVariable("Resolution", new Vector2Int(Screen.width, Screen.height))
+                () => LogVariable("Window size", $"{Screen.width} x {Screen.height}")
             ));
 
             AddCommand(Command.Create<int>(