Skip to content

Commit 0cdf893

Browse files
jay20012024claude
andcommitted
fix(savestatus): add missing .srOnly CSS rule (PR C oversight)
PR C added `<span className={styles.srOnly}>{announcement}</span>` to SaveStatus.tsx but the matching `.srOnly` rule never made it into SaveStatus.module.css — the file was added to InsertDialog's module instead. `styles.srOnly` therefore resolved to `undefined`, the span got no className, and the announcement text rendered as visible 'Draft saved' next to the SAVED status badge in the header. Standard sr-only rule restored. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
1 parent 508918c commit 0cdf893

1 file changed

Lines changed: 20 additions & 0 deletions

File tree

src/components/SaveStatus.module.css

Lines changed: 20 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -130,3 +130,23 @@
130130
display: none;
131131
}
132132
}
133+
134+
/*
135+
* Screen-reader-only announcement region. Standard sr-only pattern —
136+
* element stays in the DOM (so AT reads its content), but it's removed
137+
* from the visible layout. Added in PR C, but the matching rule was
138+
* accidentally placed in InsertDialog.module.css instead of here, so
139+
* `styles.srOnly` resolved to undefined and the announcement text
140+
* showed visibly next to the SAVED badge in the header.
141+
*/
142+
.srOnly {
143+
position: absolute;
144+
width: 1px;
145+
height: 1px;
146+
padding: 0;
147+
margin: -1px;
148+
overflow: hidden;
149+
clip: rect(0, 0, 0, 0);
150+
white-space: nowrap;
151+
border: 0;
152+
}

0 commit comments

Comments
 (0)