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

hide deleted comment if no replies #2386

Merged
merged 1 commit into from
Mar 28, 2024

Conversation

pvicensSpacedev
Copy link
Collaborator

Summary of Changes

If the comment that has been deleted has no replies then hide that comment.
Made the changes for both web and mobile

Demo or Before/After Pics

Before After
Screenshot 2024-03-27 at 13 13 06 Screenshot 2024-03-27 at 17 54 17

More pics:
Screenshot 2024-03-27 at 13 12 49

Screenshot 2024-03-27 at 17 53 24

Edge Cases

I'd just like to note the fact that if the post has 1 comment (that is deleted) then the condition to show "No comments yet" is not fulfilled and thus just shows a blank screen as in the screenshot.
I tried fixing this but am unsure how to get the comments replies in the parent component's query. It currently just checks if there are comments without ever checking if its deleted or not

Checklist

Please make sure to review and check all of the following:

  • I've tested the changes and all tests pass.
  • (if web) I've tested the changes on various desktop screen sizes to ensure responsiveness.
  • (if mobile) I've tested the changes on both light and dark modes.

Copy link

vercel bot commented Mar 27, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
gallery ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 27, 2024 9:02pm

Copy link

Bundle Sizes

Compared against 272545c

Route: No significant changes found

Dynamic import: No significant changes found

@Robinnnnn Robinnnnn merged commit 03aa4fe into main Mar 28, 2024
9 checks passed
@Robinnnnn Robinnnnn deleted the pato/fix-deleted-comments-standalone branch March 28, 2024 03:16
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