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

No return after clicking "prevent this page from creating further dialogs" #2822

Closed
michi-zuri opened this issue Oct 30, 2019 · 2 comments
Closed

Comments

@michi-zuri
Copy link

michi-zuri commented Oct 30, 2019

When testing Netlify with a potential customer (in a setting similar to a low-budget usability testing described by Steve Krug in his famous book Don't Make Me Think) I saw how the test person clicked the checkbox to prevent further dialogs. This caused her not to be able to finish a simple task: "create a new FAQ entry, then delete it".

Steps to reproduce the behavior:

  1. Go to https://cms-demo.netlify.com/
  2. Click login
  3. Click on quick add-> FAQ
  4. Write something, then click Save
  5. Now click on Delete and cancel
  6. click on Delete again and tick the checkbox to prevent further dialogs, then cancel.
  7. Try to delete the page now, a normal person will not succeed within a few minutes.

Expected behavior
No possibility to silence confirmation dialogs.

Applicable Versions:
Netlify CMS demo available here: https://cms-demo.netlify.com/
netlify-cms-app 2.9.8-beta.4
netlify-cms-core 2.13.0-beta.4
netlify-cms 2.9.8-beta.4
Browser versions: [Firefox quantum 69.0.1, Microsoft Edge, etc.]

Additional context
https://stackoverflow.com/questions/20672490/how-to-know-whether-modal-boxes-alert-prompt-confirm-have-been-disabled-i

@erezrokah
Copy link
Contributor

erezrokah commented Nov 5, 2019

Hey @michi-zuri I wasn't able to reproduce this with Chrome/Brave/Safari.
I was able to reproduce with Firefox and found this: https://superuser.com/questions/796842/disable-prevent-this-page-from-creating-additional-dialogs-for-certain-website

A possible solution is to switch to modals instead of alerts.

@martinjagodic
Copy link
Member

Closing as stale and/or outdated.

@martinjagodic martinjagodic closed this as not planned Won't fix, can't repro, duplicate, stale May 12, 2023
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

3 participants