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

"Delete" button not asking for confirmation - deleting a row #17535

Open
nunoperalta opened this issue May 19, 2022 · 9 comments
Open

"Delete" button not asking for confirmation - deleting a row #17535

nunoperalta opened this issue May 19, 2022 · 9 comments
Labels
affects/5.2 This issue or pull-request affects 5.2.x releases (and maybe further versions) affects/6.0 This issue or pull-request affects 6.0.x releases (and maybe further versions) Bug A problem or regression with an existing feature confirmed/5.2 This issue is confirmed to be reproduced on 5.2 at the time this label was set confirmed/6.0 This issue is confirmed to be reproduced on 6.0 at the time this label was set help wanted ui Issues relating to the user interface

Comments

@nunoperalta
Copy link

5.2.0

In some situations, clicking the "delete" button is deleting the row without any confirmation!!

I'm still trying to understand what's the pattern. Sometimes it asks, sometimes it deletes straight away.

sshot-2022-05-19-01-05-05-1

@williamdes
Copy link
Member

Hi @nunoperalta
Did you have such issues with 5.1.x ?

@williamdes williamdes added Bug A problem or regression with an existing feature help wanted ui Issues relating to the user interface labels May 19, 2022
@nunoperalta
Copy link
Author

Hi @nunoperalta Did you have such issues with 5.1.x ?

I don't remember ever having this issue, but it's very rare for me to be deleting rows using that button though.

@nunoperalta
Copy link
Author

nunoperalta commented Jun 18, 2022

This just happened again. All I did was open a table from the sidebar, and then using the default sort (PK descending, per my settings), I clicked the very first "x" button (which is the latest inserted row). No confirmation - the delete ran straight away. No JavaScript errors reported in DevTools.

@williamdes williamdes added affects/5.2 This issue or pull-request affects 5.2.x releases (and maybe further versions) affects/6.0 This issue or pull-request affects 6.0.x releases (and maybe further versions) labels Oct 29, 2022
@williamdes williamdes added this to the 5.2.1 milestone Oct 29, 2022
@williamdes
Copy link
Member

I managed to reproduce this !
Hit F5 to refresh the page and as soon as possible click on the delete button

@williamdes williamdes added confirmed/5.2 This issue is confirmed to be reproduced on 5.2 at the time this label was set confirmed/6.0 This issue is confirmed to be reproduced on 6.0 at the time this label was set labels Nov 20, 2022
@williamdes
Copy link
Member

I am marking this as skipped to this milestone, it may require risky changes

@williamdes williamdes removed this from the 5.2.1 milestone Nov 20, 2022
@nunoperalta
Copy link
Author

Thanks @williamdes - I still from time to time don't get the confirmation alert, and I'm pretty sure it's not a "timing issue" (like opening the page and delete quickly), but I'm glad you found a way to reproduce this anyway!

@williamdes
Copy link
Member

This one probably links to #16954

@nunoperalta
Copy link
Author

Cheers. I don't think I am seeing any warning showing, when this happens to me. Could have been a coincidence, when it happened to that user. Or just another reason for this bug to happen :)

@williamdes
Copy link
Member

Cheers. I don't think I am seeing any warning showing, when this happens to me. Could have been a coincidence, when it happened to that user. Or just another reason for this bug to happen :)

I think this could be another reason. Since the code needs to register itself to prevent the bug to occur. Anything that prevents it to register let's the bug occur

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
affects/5.2 This issue or pull-request affects 5.2.x releases (and maybe further versions) affects/6.0 This issue or pull-request affects 6.0.x releases (and maybe further versions) Bug A problem or regression with an existing feature confirmed/5.2 This issue is confirmed to be reproduced on 5.2 at the time this label was set confirmed/6.0 This issue is confirmed to be reproduced on 6.0 at the time this label was set help wanted ui Issues relating to the user interface
Projects
None yet
Development

No branches or pull requests

2 participants