feat(components): make CommentEditor configurable and use it for description #50

Closed
Christoph wants to merge 0 commits from refs/pull/50/head into main
Christoph commented 2026-09-18 10:55:28 +00:00 (Migrated from git.cbsk-tech.de)

Replace the plain textarea in CreateReviewDialog with CommentEditor bound to
the description. The dialog now passes language, disabled, rows, ariaLabel,
previewLabel and placeholder so the description field gains markdown preview
and consistent accessible labels/placeholders.

Make CommentEditor props optional and configurable:

  • onSend is now (() => void | Promise) | undefined; Enter/Cmd+Enter and
    the send button are guarded/hidden when onSend is not provided.
  • Add placeholder, ariaLabel, previewLabel and rows (default 5) to allow
    parent components to control appearance and accessibility.

Also add a small documentation tweak in commit_ai's cloud template: remind
authors to keep the title plain text and expand guidance on Markdown formatting.

Replace the plain textarea in CreateReviewDialog with CommentEditor bound to the description. The dialog now passes language, disabled, rows, ariaLabel, previewLabel and placeholder so the description field gains markdown preview and consistent accessible labels/placeholders. Make CommentEditor props optional and configurable: - onSend is now (() => void | Promise<void>) | undefined; Enter/Cmd+Enter and the send button are guarded/hidden when onSend is not provided. - Add placeholder, ariaLabel, previewLabel and rows (default 5) to allow parent components to control appearance and accessibility. Also add a small documentation tweak in commit_ai's cloud template: remind authors to keep the title plain text and expand guidance on Markdown formatting.

Pull request closed

Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
Christoph/Gitty!50
No description provided.