From de0a1e66d828f5723c4ce00957d5adae842e1a4b Mon Sep 17 00:00:00 2001
From: Shane Smiskol <shane@smiskol.com>
Date: Mon, 13 Oct 2025 13:54:50 -0700
Subject: [PATCH] software download screenshot (#36326)

* software

* clean up

* Qt ButtonControl has 0 padding

* clean up

* clean up
---
 .../ui/tests/test_ui/raylib_screenshots.py    | 22 +++++++++++++------
 system/ui/widgets/button.py                   |  2 +-
 2 files changed, 16 insertions(+), 8 deletions(-)

diff --git a/selfdrive/ui/tests/test_ui/raylib_screenshots.py b/selfdrive/ui/tests/test_ui/raylib_screenshots.py
index bd40b89624..20d0f2ecba 100755
--- a/selfdrive/ui/tests/test_ui/raylib_screenshots.py
+++ b/selfdrive/ui/tests/test_ui/raylib_screenshots.py
@@ -64,6 +64,19 @@ def setup_settings_software(click, pm: PubMaster):
   click(278, 720)
 
 
+def setup_settings_software_download(click, pm: PubMaster):
+  params = Params()
+  # setup_settings_software but with "DOWNLOAD" button to test long text
+  params.put("UpdaterState", "idle")
+  params.put_bool("UpdaterFetchAvailable", True)
+  setup_settings_software(click, pm)
+
+
+def setup_settings_software_release_notes(click, pm: PubMaster):
+  setup_settings_software(click, pm)
+  click(588, 110)  # expand description for current version
+
+
 def setup_settings_firehose(click, pm: PubMaster):
   setup_settings(click, pm)
   click(278, 845)
@@ -106,18 +119,14 @@ def setup_homescreen_update_available(click, pm: PubMaster):
   close_settings(click, pm)
 
 
-def setup_software_release_notes(click, pm: PubMaster):
-  setup_settings(click, pm)
-  setup_settings_software(click, pm)
-  click(588, 110)  # expand description for current version
-
-
 CASES = {
   "homescreen": setup_homescreen,
   "settings_device": setup_settings,
   "settings_network": setup_settings_network,
   "settings_toggles": setup_settings_toggles,
   "settings_software": setup_settings_software,
+  "settings_software_download": setup_settings_software_download,
+  "settings_software_release_notes": setup_settings_software_release_notes,
   "settings_firehose": setup_settings_firehose,
   "settings_developer": setup_settings_developer,
   "keyboard": setup_keyboard,
@@ -125,7 +134,6 @@ CASES = {
   "offroad_alert": setup_offroad_alert,
   "homescreen_update_available": setup_homescreen_update_available,
   "confirmation_dialog": setup_confirmation_dialog,
-  "software_release_notes": setup_software_release_notes,
 }
 
 
diff --git a/system/ui/widgets/button.py b/system/ui/widgets/button.py
index 141f682db0..ef28df8112 100644
--- a/system/ui/widgets/button.py
+++ b/system/ui/widgets/button.py
@@ -86,7 +86,7 @@ class Button(Widget):
                button_style: ButtonStyle = ButtonStyle.NORMAL,
                border_radius: int = 10,
                text_alignment: TextAlignment = TextAlignment.CENTER,
-               text_padding: int = 20,
+               text_padding: int = 0,
                icon=None,
                multi_touch: bool = False,
                ):