Yes, popups should be a direct result of a user action. Doing them in ajax callback will not do the trick. Also, using async:false
is bad – in FF it is known to block the whole browser. Think of some other way to do the check:
- it could be the first thing you do in the popup
- you can open the popup on click and manipulate it later when the callback fires
- you can require the user to click again some button to trigger the popup (probably the worst solution)
- you can do it on page load