fix: refresh blog layouts and typography#91
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting
| <span | ||
| className={cn( | ||
| 'relative z-10 text-center transition-colors duration-200 ease-out', | ||
| checked ? 'text-black' : 'text-white drop-shadow-[1px_1px_0px_rgba(0,0,0,0.45)]', | ||
| )} |
There was a problem hiding this comment.
Restore toggle’s state-specific accessible label
Rendering both offLabel and onLabel spans at the same time leaves the switch’s accessible name static (screen readers will always encounter both labels regardless of aria-checked). Without the visible text changing or an alternative aria-label, users relying on assistive tech cannot tell whether the current sort is “Latest” or “Popular”. Consider only exposing the active label or updating the accessible name when checked toggles so that the spoken state matches the visual state.
Useful? React with 👍 / 👎.
Summary
Testing
https://chatgpt.com/codex/tasks/task_e_68e7790d1bf8832db54abd306916a0df