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

Feature request: erase previous comments when alt-text is filled #49

Closed
scarf005 opened this issue Aug 2, 2024 · 1 comment
Closed
Assignees

Comments

@scarf005
Copy link

scarf005 commented Aug 2, 2024

hi, i stumbled across this bot while opening an issue reqeust on typescript-eslint/typescript-eslint#9699.

Status quo

  • the bot will run on every issue body creation/modification, and will add a new comment on alt-text violation.
  • even after all the alt-text violation is fixed, the comments are still there unmodified.

The issue

an issue with two bot warning comment remaining despite the error being fixed after

  • it feels awkward and crowded to keep having the comments even after all the alt-text issues are fixed.
  • in this screenshot, the second comment is automatically hidden (due to duplicate entry?), but the first isn't.
  • this may be confusing considering the alt-text is now fixed.

The suggestion

it'd be great to have an option to erase/hide previous accessibility-alt-text-bot comments when there's no alt-text issue found.

@kendallgassner
Copy link
Collaborator

Released in v1.6.0

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

No branches or pull requests

2 participants