mirror of
https://github.com/ARMSX2/ARMSX2.git
synced 2026-08-24 16:50:16 -07:00
J1coding
be72a8e1eb
It shipped as its own Section on the settings page, which put it below the parameter list. With crt-aperture selected that is twenty-two sliders and about thirty-nine swipes, on the one control a tester had asked for by name. Found by walking the screen in the simulator rather than by reading it, which is the only way this kind of thing turns up. It moves into ShaderChainSection, directly under Preset and above Install Shader Pack, so the three ways to get a preset sit together in the order you would try them: pick one you have, download one, install one from a file. That also puts it in the in-game pause panel, which is a gain rather than a side effect, and it works only because GameScreenView wraps the shared section in its own NavigationStack. Without that every NavigationLink in the section is dead on tap, Preset included, so the fence now checks for the stack in the text immediately around the mount. Immediately around, and not anywhere earlier in the file, because the first version of that check searched backwards from the mount through the whole of GameScreenView and any one of its several other NavigationStacks satisfied it. Deleting the one that matters left the suite green. That is the third time in this branch an assertion has been satisfied by a neighbour, and the only reason any of the three were caught is that each new check was run against a deliberately broken source before being trusted. The row sits outside the enabled gate, which the first attempt at this move got wrong. Every other row in that section is behind `if enabled`, and putting the download row there too hid it whenever the chain was off -- so a first run had nothing to select, no way to fetch anything, and no hint that the toggle came first. Caught by relaunching with the chain off and looking, one screenshot after the change built.