Cancel button calls resolve

In the ‘Are you sure?’ examples, the cancel button should call reject, not resolve (Proper promise syntax, I think)

[limonte]

Thank you @AmitMY for standing for you point, I probably have to agree with you and make this breaking change 😃

Please give your opinion about this structure:

swal({
  ...
}).then(function(result) {
  // handle confirm, result is needed for modals with input
}, function(dismiss) {
  // dismiss can be "cancel" | "close" | "outside"
});

cc @samturrell @Tonkpils @jdcrensh

[AmitMY]

In my opinion, because all do the same action, they should do reject() That way, if there is a promise in the SWAL, and it rejects, it will normally reject(error), such that you can handle by checking if(error) And if not, and the promise sends reject() it means just ‘close’ - no nead for error handling.

This removes the check if(isConfirm), and uses the build in promise rejection

What do you think? Where am I logically failing?