Attention rows and the provisioning summary lose the spaces around "·" #112

Closed
opened 2026-10-09 19:55:37 -04:00 by goddard · 1 comment
Collaborator

Two separators render without their spaces, so text runs together.

  • +page.svelte:413 has <span class="evidence-separator"> · </span>. Svelte trims whitespace at the edges of an element's text, so Servers attention rows read "...status check.·Evidence 1m 55s ago". Phones hide this separator (+page.svelte:1073), so it shows on wider screens.
  • provisioning/+page.svelte:367 has <span> · {queueSummary}</span>, which renders as "3 active· 2 queued".

ui-patterns.md:111-113 already says to keep separators inside string expressions for this reason.

Screenshots

Before:
Before

After (mockup):
After (mockup)

Fix

  • Write both separators as string expressions, e.g. {' · '}
Two separators render without their spaces, so text runs together. - `+page.svelte:413` has `<span class="evidence-separator"> · </span>`. Svelte trims whitespace at the edges of an element's text, so Servers attention rows read "...status check.·Evidence 1m 55s ago". Phones hide this separator (`+page.svelte:1073`), so it shows on wider screens. - `provisioning/+page.svelte:367` has `<span> · {queueSummary}</span>`, which renders as "3 active· 2 queued". `ui-patterns.md:111-113` already says to keep separators inside string expressions for this reason. **Screenshots** Before: ![Before](https://gitea.thewrightserver.net/attachments/470d1dec-007f-428c-9004-8a07c75cdd66) After (mockup): ![After (mockup)](https://gitea.thewrightserver.net/attachments/16c53212-2e88-40e1-ae83-fd7453761fb7) **Fix** - [x] Write both separators as string expressions, e.g. `{' · '}`
goddard added the
priority
low
type/bug
labels 2026-10-09 19:55:37 -04:00
josh was assigned by goddard 2026-10-09 19:55:37 -04:00
Author
Collaborator

Reviewed b9bc897d6e against current HEAD 4501eba873. Both separators preserve their spaces, so I'm closing this.

Checklist

  • Use string expressions for both separators. The Servers attention row and provisioning summary both use {' · '}. Rendering the exact changed markup produces “Last status check. · Evidence 1m 55s ago” and “3 active · 2 queued”. An empty queue still omits the separator, and multiple queue states retain their spacing.

Review notes

  • All four isolated SSR checks of the source snippets passed. Both complete page components compiled without warnings. These were renderer checks rather than browser screenshots.
  • The unchanged phone CSS still hides the attention separator and places evidence on its own line.
  • Shared type/style and lint checks passed. The relevant fleet and language tests passed as part of the combined #63/#98/#112 validation: 403 tests across 17 files, after successfully rerunning the recovery test whose child process was initially blocked by the sandbox.
  • Converted the existing Fix bullet to a completed checkbox while preserving its wording.
Reviewed b9bc897d6e6f1490a8b82ac1954811760b6c320e against current HEAD 4501eba8735ceacccc924138502ed424b3310020. Both separators preserve their spaces, so I'm closing this. **Checklist** - [x] **Use string expressions for both separators.** The Servers attention row and provisioning summary both use `{' · '}`. Rendering the exact changed markup produces “Last status check. · Evidence 1m 55s ago” and “3 active · 2 queued”. An empty queue still omits the separator, and multiple queue states retain their spacing. **Review notes** - All four isolated SSR checks of the source snippets passed. Both complete page components compiled without warnings. These were renderer checks rather than browser screenshots. - The unchanged phone CSS still hides the attention separator and places evidence on its own line. - Shared type/style and lint checks passed. The relevant fleet and language tests passed as part of the combined #63/#98/#112 validation: 403 tests across 17 files, after successfully rerunning the recovery test whose child process was initially blocked by the sandbox. - Converted the existing Fix bullet to a completed checkbox while preserving its wording.
Sign in to join this conversation.