Skip to content

Add buttons to hide comments (off-topic, outdated, resolved) - #374

Open
cead22 wants to merge 8 commits into
mainfrom
carlos-buttons
Open

Add buttons to hide comments (off-topic, outdated, resolved)#374
cead22 wants to merge 8 commits into
mainfrom
carlos-buttons

Conversation

@cead22

@cead22 cead22 commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

This makes it easier to clean up PRs and GitHub issues by hiding bot comments and comments that are no longer relevant

Tested using one of the buttons in one of my PRs
image

Buttons don't show on PR description
image

Resolved comment before refresh
image

Resolved comment after refresh
image

@cead22
cead22 requested a review from roryabraham August 14, 2026 13:58
@cead22 cead22 self-assigned this Aug 14, 2026
@cead22
cead22 marked this pull request as ready for review August 14, 2026 13:59
Comment thread src/js/lib/api.js Outdated
Comment thread src/js/lib/hideCommentButtons.js Outdated
Comment thread src/js/lib/hideCommentButtons.js Outdated
@cead22

This comment has been minimized.

@roryabraham

This comment has been minimized.

@roryabraham roryabraham left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fews bugs:

  1. When hiding a comment, it disappears entirely:
Expected Actual
Image Image
  1. It should not show on the PR description:
Image
  1. It doesn't look like reasons are being recorded as expected:
Expected Actual (after refresh)
Image Image

@cead22

This comment has been minimized.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants