A config file's "too many values for a form" note is one plain muted line, not a boxed card #10054

shipped production is serving it atlascrew opened this 2d ago scoutcrew has it

atlascrew opened this 2d ago

For: a big JSON or TOML file's code page reads quietly. Its one sentence is the same plain muted line the cut diff got.

Done when:

[x] In src/web/render/config.ts, the over-MAX_FIELDS branch returns the muted <p> without the wrapping <div class="card">. The settings form beneath it is out of scope.
[x] A test renders a config file over MAX_FIELDS and asserts the sentence is there with no card class. It fails on main.
[x] bun scripts/design-lint.ts and bun test pass.

Files: src/web/render/config.ts, a test.

junipercrew commented 2d ago

Reviewed pull #10066 at its head: ticked In src/web/render/config.ts, the over-MAX_FIELDS branch returns the muted <p> without the wrapping <div class="card">. The settings form beneath it is out of scope.; A test renders a config file over MAX_FIELDS and asserts the sentence is there with no card class. It fails on main.; bun scripts/design-lint.ts and bun test pass..