Skip to content

CS-213 Comment button stays greyed out with numbered formatting#2368

Open
github-actions[bot] wants to merge 2 commits intomainfrom
chas/comment-disabled-error
Open

CS-213 Comment button stays greyed out with numbered formatting#2368
github-actions[bot] wants to merge 2 commits intomainfrom
chas/comment-disabled-error

Conversation

@github-actions
Copy link
Copy Markdown
Contributor

This is an automated pull request to merge chas/comment-disabled-error into dev.
It was created by the [Auto Pull Request] action.

@vercel
Copy link
Copy Markdown

vercel bot commented Mar 25, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
app Ready Ready Preview, Comment Mar 30, 2026 9:29pm
comp-framework-editor Ready Ready Preview, Comment Mar 30, 2026 9:29pm
portal Ready Ready Preview, Comment Mar 30, 2026 9:29pm

Request Review

@cursor
Copy link
Copy Markdown

cursor bot commented Mar 25, 2026

PR Summary

Low Risk
Low risk UI logic change confined to client-side comment validation; main risk is edge cases where empty structured nodes are misclassified as content.

Overview
Fixes comment submission enabling by updating CommentForm’s hasContent check to recursively detect meaningful TipTap content (including nested text and mentions), instead of only scanning top-level paragraph nodes. This prevents the Comment button from staying disabled when content is present via structured formatting (e.g., lists/numbered formatting).

Written by Cursor Bugbot for commit e954209. This will update automatically on new commits. Configure here.

@chasprowebdev chasprowebdev changed the title [dev] [chasprowebdev] chas/comment-disabled-error CS-213 Comment button stays greyed out with numbered formatting Mar 25, 2026
@linear
Copy link
Copy Markdown

linear bot commented Mar 25, 2026

@chasprowebdev
Copy link
Copy Markdown
Contributor

Recording

comment.mov

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants