fix(frontend): confine the modal restyle to the wizard, and trim the comments

The wider side padding and lighter dialog heading were changing all 17 Modal2
dialogs to suit this one flow. They move behind an opt-in `formStyling`, taken
by the three dialogs this branch owns; every other Modal2 renders as it did.

Also drops two comments that cited a design approval rather than a constraint,
and shortens the blocks that had grown past the four lines AGENTS.md asks for.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
This commit is contained in:
Guilhem Lemouel
2026-08-18 10:31:21 +02:00
co-authored by Claude Opus 5
parent b89d506b91
commit 0b33bb67c6
10 changed files with 51 additions and 77 deletions
@@ -76,14 +76,10 @@ const REPRESENTABLE_PARAMS = ['sslmode']
const COSMETIC_PARAMS = ['application_name']
/**
* The name of a parameter this string carries that the resource cannot honour.
*
* An allowlist rather than a list of known-bad names: libpq keeps adding parameters, and the
* ones that matter most are the ones that would be missed. `sslrootcert=system` demands full
* certificate verification, `channel_binding=require` demands SCRAM binding,
* `target_session_attrs=read-write` decides which node is acceptable, `options=-csearch_path=…`
* decides which schema tables land in. Dropping any of them saves a connection weaker or
* simply other than the one that was pasted, behind a probe that reports success.
* The name of a parameter this string carries that the resource cannot honour. An allowlist,
* not a list of known-bad names: libpq keeps adding parameters, and the ones that matter are
* the ones that would be missed. Dropping one silently saves a connection weaker or simply
* other than the one pasted, behind a probe that reports success.
*/
export function unsupportedConnectionParam(connectionString: string): string | undefined {
const query = connectionString.split('?').slice(1).join('?')