there was an error with mrujs.confirm

[karn18]

Expected Behaviour

Should show confirm dialog after clicked destroy button

Actual Behaviour

It shows an error on console
there was an error with mrujs.confirm

Steps to Reproduce

• create new blogs
• try to delete one of theme

[KonnorRogers]

Can you confirm that you can use window.confirm("message") on this page?

https://github.com/ParamagicDev/mrujs/blob/e83c192b82df25f60a3a3b796e95db698139c9ec/src/mrujs.ts#L164

https://github.com/ParamagicDev/mrujs/blob/e83c192b82df25f60a3a3b796e95db698139c9ec/src/confirm.ts#L47

In the meantime, I'll create a release which actually logs the error instead of saying there was an error and not showing the error.

[karn18]

After trying to call window.confirm("hello") on inspector console, It throws error Uncaught InternalError: too much recursion

[KonnorRogers]

Do you have a repo I could look at? This looks like something is overriding window.confirm.

[karn18]

The repo is here !!!

https://github.com/karn18/abc

[KonnorRogers]

@karn18 is this still an issue in the latest v0.8.3? Apologies for leaving you hanging this long.

[karn18]

@karn18 is this still an issue in the latest v0.8.3? Apologies for leaving you hanging this long.

@ParamagicDev Now it works well after update to latest version. Thank you.