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

Display confirmation box when closing an issue with a comment #3884

Closed
Richienb opened this issue Jan 10, 2021 · 9 comments
Closed

Display confirmation box when closing an issue with a comment #3884

Richienb opened this issue Jan 10, 2021 · 9 comments

Comments

@Richienb
Copy link
Contributor

Richienb commented Jan 10, 2021

Useful to avoid accidentally closing issues.

Should be applied for all issue pages: https://github.com/*/*/issues/*

Should be triggered when clicking this button:

Close with comment

@Richienb Richienb changed the title Display confirmation box when closing and commenting an issue Display confirmation box when closing an issue with a comment Jan 10, 2021
@yakov116
Copy link
Member

Don't we already put the close button on the other side to avoid this issue?

@fregante

This comment has been minimized.

@fregante fregante marked this as a duplicate of #1037 Jan 10, 2021
@Richienb

This comment has been minimized.

@fregante fregante marked this as not a duplicate of #1037 Jan 11, 2021
@fregante

This comment has been minimized.

@fregante
Copy link
Member

Should be triggered when clicking this button:

Close with comment

I did this accidentally a few times, but I don't know if I'd find the confirmation less annoying than just reopening the issue 🤔

Don't we already put the close button on the other side to avoid this issue?

Yes: https://github.com/sindresorhus/refined-github/blob/382b398cbaccbb5a3f7135cf873f1302daf5901e/source/features/safer-destructive-actions.css

@Richienb
Copy link
Contributor Author

Perhaps, instead, there should be a 2-second countdown that allows you to immediately cancel the action.

@fregante
Copy link
Member

Given the non-destructiveness of the action I don't think it's worth messing with that. That button already has enough logic behind it (and more is coming with #3881)

@fregante
Copy link
Member

As annoying as it is (honestly, just a little), I think having to confirm your action even when you want to close it actually makes it worse.

Now:

  • intentional: 1 click
  • unintentional: 2 clicks

With a confirmation

  • intentional: 2 clicks
  • unintentional: 2 clicks

🤷‍♂️

@sgreene570
Copy link

sgreene570 commented Nov 3, 2022

In some cases, closing an issue accidentally can have unintended consequences, such as triggering (the start or end of) different github actions. It can also lead to confusion among PR reviewers, especially if GH events are pushed to some external source (slack channels, etc).

@fregante would it be possible to re-open this issue?

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

No branches or pull requests

4 participants