Skip to content

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

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

CS-213 Comment button stays greyed out with numbered formatting#2368
github-actions[bot] wants to merge 1 commit 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 25, 2026 3:20am
comp-framework-editor Ready Ready Preview, Comment Mar 25, 2026 3:20am
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
portal Skipped Skipped Mar 25, 2026 3:20am

Request Review

@cursor
Copy link
Copy Markdown

cursor bot commented Mar 25, 2026

PR Summary

Low Risk
Low risk: only changes client-side detection of whether a TipTap comment is empty, affecting when the submit button is enabled and when submits are blocked.

Overview
Updates CommentForm’s hasContent check to recursively detect non-empty TipTap content (including nested text and mentions), instead of only scanning top-level paragraph nodes.

This changes when the comment submit action/button is considered enabled, reducing false “empty comment” cases.

Written by Cursor Bugbot for commit ae973c3. 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.

1 participant