Nested discussion, sorted card feed, review resolve thread, and mentions with reactions.
Comments
4- SJSarah Jenkins
The density feels right for docs feedback. Sorting by Top is useful during launch week.
- ARAlex Rivera
Flat threads are easier to scan when reviewing before a release.
- MGMaria Garcia
Could we pin FAQ answers, or is Top sort enough for that use case?
- DCDavid Chen
Looks good. Keep timestamps LTR in RTL layouts.
Review comments
2 openLine-anchored feedback on this change.
- SJSarah JenkinsOpen
button.tsx:46
Default height should stay at 46px on mobile — don’t shrink below the control scale.
ARAlex Rivera12m agoAgreed — keeping `h-[46px]` and only adjusting padding.
- DCDavid ChenOpen
input.tsx:28
Focus ring looks good — can we match the shared `--ring` token?
Discussion
MentionsHey @alex — can you glance at the mention chips before we merge?
On it — will leave notes in the PR.
Ship looks solid. @david thoughts on the reaction row density?
Discussion
6This is a solid approach for accessible dialogs. Also consider `inert` on the rest of the page while the modal is open — it’s supported in modern browsers now.
Great point — we added `inert` in v1.2 alongside the focus trap so screen readers stay inside the dialog.
Is `inert` preferable to `aria-hidden` on the root? We’ve been using the latter.
The CSS variables comparison was useful. Moving our theme to custom properties cut about 14 KB from the JS bundle.