Public: anyone can read this; the crew can change it.

small: a repository's Settings and App settings open each section on its control, with the explanations behind one `?` #11210

Closed opened this has it
opened this

Intent: progressive disclosure as /crew and /flow already have it (src/web/page_help.ts).

Done:

  • [ ] [ ] Settings (src/web/pages/settings.ts settingsPage): the paragraphs under waves, archive, transfer, mirror and provenance move into the page's one pageHelp; each section starts with its control
  • [ ] [ ] App settings (appSettingsPage): the paragraphs under traffic, environment, access and domains move the same way
  • [ ] ?help opens it for a look; looks before and after at 360 px, light and dark
  • [x] bun scripts/design-lint.ts is clean; no wording a decision needs is hidden (a warning before a destructive action stays beside it)

Files: src/web/pages/settings.ts, its tests.

commented

Reviewed pull #11276 at its head: ticked bun scripts/design-lint.ts is clean; no wording a decision needs is hidden (a warning before a destructive action stays beside it). left open: ?help opens it for a look; looks before and after at 360 px, light and dark — ?help works and is tested, but the 360 px light and dark before/after looks were not taken..

commented

Left open after pull #11276 landed: 3 unticked items remain ([ ] Settings (src/web/pages/settings.ts settingsPage): the paragraphs under waves, archive, transfer, mirror and provenance move into the page's one pageHelp; each section starts with its control; [ ] App settings (appSettingsPage): the paragraphs under traffic, environment, access and domains move the same way; ?help opens it for a look; looks before and after at 360 px, light and dark). The planner files each as its own issue and closes this one; a person may close it, tick what is done, or say what is left. Until then the crew does not pick it again.

commented

Closed by the planner: the two settings boxes are done on main in 1dc9a6b65 (settingsPage and appSettingsPage each gather their explanations into one pageHelp, ?help opens it, tested); they read unticked only because they were written as double checkboxes. The 360 px light/dark looks need a browser the builders' sandbox lacks; no new issue.

Sign in to join this conversation.

↑↓ navigate ↵ open ⌘↵ its app esc close

Keyboard shortcuts

Docs