Improve manual option for scanline count

drhelius · drhelius · commit 74ceef57f460 · 2025-05-02T00:18:02.000+02:00
diff --git a/platforms/shared/desktop/gui_menus.cpp b/platforms/shared/desktop/gui_menus.cpp
@@ -461,20 +461,22 @@ static void menu_video(void)
 
             if (config_video.scanline_mode == 2)
             {
-                ImGui::PushItemWidth(200.0f);
-                if (ImGui::SliderInt("##scanline_start", &config_video.scanline_start, 0, 241, "Start = %d"))
+                ImGui::Separator();
+                ImGui::TextDisabled("Displaying %d scanlines:", MAX(0, config_video.scanline_end - config_video.scanline_start + 1));
+                ImGui::PushItemWidth(250.0f);
+                if (ImGui::SliderInt("##scanline_start", &config_video.scanline_start, 0, 241, "Start line = %d"))
                 {
                     emu_set_scanline_start_end(
                     config_debug.debug ? 0 : config_video.scanline_start,
                     config_debug.debug ? 241 : config_video.scanline_end);
                 }
-                if (ImGui::SliderInt("##scanline_end", &config_video.scanline_end, 0, 241, "End = %d"))
+                if (ImGui::SliderInt("##scanline_end", &config_video.scanline_end, 0, 241, "End line = %d"))
                 {
                     emu_set_scanline_start_end(
                     config_debug.debug ? 0 : config_video.scanline_start,
                     config_debug.debug ? 241 : config_video.scanline_end);
                 }
-                if (ImGui::Button("Show all scanlines"))
+                if (ImGui::Button("Show all scanlines", ImVec2(250.0f, 0)))
                 {
                     if (!config_debug.debug)
                     {

Original file line number	Diff line number	Diff line change
`@@ -461,20 +461,22 @@ static void menu_video(void)`
`461`	`461`
`462`	`462`	`if (config_video.scanline_mode == 2)`
`463`	`463`	`{`
`464`		`- ImGui::PushItemWidth(200.0f);`
`465`		`- if (ImGui::SliderInt("##scanline_start", &config_video.scanline_start, 0, 241, "Start = %d"))`
	`464`	`+ ImGui::Separator();`
	`465`	`+ ImGui::TextDisabled("Displaying %d scanlines:", MAX(0, config_video.scanline_end - config_video.scanline_start + 1));`
	`466`	`+ ImGui::PushItemWidth(250.0f);`
	`467`	`+ if (ImGui::SliderInt("##scanline_start", &config_video.scanline_start, 0, 241, "Start line = %d"))`
`466`	`468`	`{`
`467`	`469`	`emu_set_scanline_start_end(`
`468`	`470`	`config_debug.debug ? 0 : config_video.scanline_start,`
`469`	`471`	`config_debug.debug ? 241 : config_video.scanline_end);`
`470`	`472`	`}`
`471`		`- if (ImGui::SliderInt("##scanline_end", &config_video.scanline_end, 0, 241, "End = %d"))`
	`473`	`+ if (ImGui::SliderInt("##scanline_end", &config_video.scanline_end, 0, 241, "End line = %d"))`
`472`	`474`	`{`
`473`	`475`	`emu_set_scanline_start_end(`
`474`	`476`	`config_debug.debug ? 0 : config_video.scanline_start,`
`475`	`477`	`config_debug.debug ? 241 : config_video.scanline_end);`
`476`	`478`	`}`
`477`		`- if (ImGui::Button("Show all scanlines"))`
	`479`	`+ if (ImGui::Button("Show all scanlines", ImVec2(250.0f, 0)))`
`478`	`480`	`{`
`479`	`481`	`if (!config_debug.debug)`
`480`	`482`	`{`