diff --git a/.agents/skills/frontend/SKILL.md b/.agents/skills/frontend/SKILL.md index 0c59ef6301..1d8b6ee440 100644 --- a/.agents/skills/frontend/SKILL.md +++ b/.agents/skills/frontend/SKILL.md @@ -286,3 +286,45 @@ Grep `stage="preview"` and `stage="beta"` and delete every match: - Inline `` usages — delete the element and unwrap the `flex items-center gap-2` div There's no other cleanup. The component itself stays in place for the next pre-GA feature. + +### Drawer Component Usage + +`DrawerContent` requires `DrawerTitle` (and optionally `DrawerDescription`) inside it. Omitting them generates a console error and breaks screen reader accessibility. `DrawerHeader` and `DrawerFooter` are optional layout wrappers. + +```tsx + + Open + + Session Details + Viewing trace for this chat. + {/* content */} + + +``` + +If the title is visually redundant, hide it from sighted users while keeping it for screen readers: + +```tsx +Details +``` + +### Dialog Component Usage + +`DialogContent` requires `DialogTitle` (and optionally `DialogDescription`) inside it. Omitting them generates a console error and breaks screen reader accessibility. `DialogHeader` and `DialogFooter` are optional layout wrappers. + +```tsx + + Open + + Confirm Action + This cannot be undone. + {/* content */} + + +``` + +If the title is visually redundant, hide it from sighted users while keeping it for screen readers: + +```tsx +Details +```