Detected the use of
alert, confirm and prompt JS-0052
Anti-pattern
Minor
5 months ago
—
2 years old
Unexpected alert
100 },
101 function (error) {
102 console.log("FAILED...", error);
103 alert("Form Submission Failed! Try Again");104 },
105 );
106});Unexpected alert
96 function (response) {
97 console.log("SUCCESS!", response.status, response.text);
98 document.getElementById("contact-form").reset();
99 alert("Form Submitted Successfully");100 },
101 function (error) {
102 console.log("FAILED...", error);
Description
JavaScript's alert, confirm and prompt functions are widely considered to be obtrusive as UI elements and should be replaced by a more appropriate custom UI implementation.
Furthermore, alert is often used while debugging code, which should be removed before deployment to production.
Bad Practice
alert("here!");
confirm("Are you sure?");
prompt("What is your name?", "John Doe");
Recommended
customAlert("Something happened!");
customConfirm("Are you sure?");
customPrompt("Who are you?");
function foo() {
const alert = myCustomLib.customAlert;
alert();
}