[tomjemmett]

resolves #33

• only show relevant checkboxes, rather than disabling them
• hide the naming guidance if a valid scenario name has been entered
• make the guidance link appear after the validation messages
• change the colour of the validation messages to red

[Copilot]

🟡 Changes recommended

It introduces a new undeclared {withr} dependency in server logic and uses a likely-unsupported bslib theme variable, both of which risk the intended behavior not working in deployment.

Get a fresh assessment by requesting another Copilot review.

Pull request overview

This PR updates the “Scenario name” UX to reduce confusion around unavailable scenario actions and to improve validation/guidance presentation in the Shiny UI.

Changes:

• Only presents scenario-type radio options when they’re actually available (instead of disabling them).
• Reorders the “Start” button vs naming guidance content and aligns the button without extra UI wrapper markup.
• Attempts to theme validation message color via bslib theme variables.

File summaries

File	Description
R/app_ui.R	Simplifies scenario-type radios, reorders start button vs naming guidance, and adds a theme variable intended to make validation messages red.
R/app_server.R	Adjusts validation-side show/hide behavior for naming guidance and updates radio choices dynamically based on whether saved scenarios exist.

Review details

• Files reviewed: 2/2 changed files
• Comments generated: 3
• Review effort level: Lite

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

[Lextuga007]

I like this approach, thank you. Only change is that the text colour as we still need to avoid red but the use of symbols like the triangle with exclamation with text, perhaps bold, would be more accessible.

[tomjemmett]

you are somewhat limited to what you can achieve with shiny validation though, at least you may have to spend a lot of time unpicking the classes and figuring out what to change. shiny validation does append aria-live="polite" [docs] to the container

[tomjemmett]

achieved this with a bit of css:

ideally we would utilise aria-describedby, but I don't think we will get that level of control.

[Lextuga007]

Works ok locally and did a build check all passes. Thanks for looking into ways of supporting the text using CSS, much appreciated.