Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

preview-hidden-comments show comment preview even if reason is unknown #6098

Closed
Katsute opened this issue Oct 23, 2022 · 0 comments · Fixed by #6113
Closed

preview-hidden-comments show comment preview even if reason is unknown #6098

Katsute opened this issue Oct 23, 2022 · 0 comments · Fixed by #6113

Comments

@Katsute
Copy link
Contributor

Katsute commented Oct 23, 2022

Description

As mentioned in #6017 some hidden comments don't have the reason included, meaning the comment would remain hidden.

Could this be changed so it would show the comment in all cases, even with no reason?

It seems like this could be implemented by removing:

if (!reason) {
return;
}

and only adding the reason label if it is not null:

<span className="Label mr-2">{upperCaseFirst(reason)}</span>{commentText.slice(0, 100)}

Screenshot

No response

Example URLs

#3647

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

Successfully merging a pull request may close this issue.

1 participant