From d73d673508e143f2837de658144464fec72b1c4f Mon Sep 17 00:00:00 2001 From: Abdessamad Derraz <3028866+Abdess@users.noreply.github.com> Date: Mon, 9 Nov 2026 06:33:52 +0100 Subject: [PATCH] docs: say where the systems list opens The page said a long list opens on None. The list opens on its first system since the three tick lists share one rule (A on None unticked every system as a beginner's first press). The page now says what the screen does, and a test reads both. --- tests/test_install_tui.py | 16 ++++++++++++++++ wiki/installer.md | 10 +++++----- 2 files changed, 21 insertions(+), 5 deletions(-) diff --git a/tests/test_install_tui.py b/tests/test_install_tui.py index 9431b4a78..660478b42 100644 --- a/tests/test_install_tui.py +++ b/tests/test_install_tui.py @@ -3812,6 +3812,22 @@ class BackTicksAndSaveFolder(unittest.TestCase): text = " ".join(install.tui_view(state, 80, 24)) self.assertNotIn("No system is ticked", text) + def test_the_page_says_where_the_list_opens(self): + """wiki/installer.md said a long list opens on None, where A + unticked every system as the first press: the focus moved to the + first system, and the page kept the old words.""" + state = self.long_list() + self.assertGreater(len(state.systems), install.TUI_LONG_LIST) + press(state, "down", "enter") + rows = install.tui_rows(state) + self.assertEqual(state.focus["systems"], + next(i for i, row in enumerate(rows) if row.action == "system")) + self.assertEqual([row.action for row in rows][-3:], ["all", "none", "done"]) + page = " ".join((REPO_ROOT / "wiki" / "installer.md").read_text(encoding="utf-8").split()) + self.assertNotIn("opens on **None**", page) + self.assertIn("It opens on its first system, every system ticked", page) + self.assertIn("**None** (`a` and `n`) sit at the end of the list", page) + def test_back_after_a_choice_keeps_the_choice(self): state = self.long_list() press(state, "down", "enter", "enter") diff --git a/wiki/installer.md b/wiki/installer.md index 894674a5d..d9812ee66 100644 --- a/wiki/installer.md +++ b/wiki/installer.md @@ -166,11 +166,11 @@ how many files are missing, how much they weigh, how many are in place and how much space is free, before anything is written. The other choices sit one press away. **Choose systems** ticks the consoles you -play, with the total to download under the list. A long list opens on **None**: -A or Enter there unticks every system, and A on a system then takes that one alone. A -on a system (**Continue**) always opens the confirm screen, which counts the choice -before **Install**: it never installs by itself. Back with nothing ticked keeps the ticks -the list opened with. **Options** sets the +play, with the total to download under the list. It opens on its first system, +every system ticked: A or Enter ticks or unticks the one in focus, and **All** and +**None** (`a` and `n`) sit at the end of the list. B or Esc (**Done**) takes the +choice back to the plan, which counts it before **Install**: the list never installs +by itself, and Done with nothing ticked is refused. **Options** sets the preferred regions, game data, the installed cores only, the cores, the parallel downloads, the folder and, for a list with files read from the frontend's save directory, that directory. Each row names the option it sets, and **Show the