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

The live status line draws §5's badge, and .live-status leaves the markup #4251

Merged opened this

Check · Review · Land · Live done — Landed and live in production.

Merged

scout/forge:scout/4240-the-live-status-line-draws-5-s-badge-and → main 1 commit fork

  • Merged. scout/4240-the-live-status-line-draws-5-s-badge-and was merged into main as 6aba9e4.

Review

Sign in to review this change.
2 files changed +12 −1
src/web/live.ts +1 −1
@@ -65,7 +65,7 @@ export function liveMount(config: LiveConfig): Html {
6565 data-app="${config.app ? "1" : ""}"
6666 ></div>
6767 <p
68 class="live-status"
68 class="sr-only"
6969 id="live-status"
7070 role="status"
7171 aria-live="polite"
test/live-pages.test.ts +11
@@ -281,3 +281,14 @@ describe("the beat of a region with data-live-every", () => {
281281 expect(cleared.length).toBe(0);
282282 });
283283 });
284
285 describe("the live mount's markup", () => {
286 test("the status line is a polite region drawn with §5's .sr-only, and the pill keeps its words and link", async () => {
287 const { liveMount } = await import("../src/web/live.ts");
288 const out = String(liveMount({ repo: "/a/b" }));
289 expect(out).not.toContain('class="live-status"');
290 expect(out).toMatch(/<p\s+class="sr-only"\s+id="live-status"\s+role="status"\s+aria-live="polite"/);
291 expect(out).toContain('id="live-pill"');
292 expect(out).toContain("New activity — refresh");
293 });
294 });

↑↓ navigate ↵ open ⌘↵ its app esc close

Keyboard shortcuts

Docs