That people are actually still willing to do this in 2019 with BTC above 10K and ETH above 300 always surprises me, even if it's for a toy example like this article (as it might incite other less tech-savy people to believe it's ok to do something like that).
If that were the case, the security of the private key would be compromised. But the website is client side only.
Or maybe the complaint is that non tech savvy users might be getting confused and might be too trustful to other sites that do private key handling server side.
This is why most crypto web-apps have a browser-extension form; you can check in your browser extension list to verify which version of the extension you have, and so know that it’s safe if someone you trust has already audited that version.
If it's a specific browser extension or a web page that you can run after you've cut the internet doesn't really make a difference. Note that vanity-eth.tk points out that you don't have to trust them and how to ensure that the private key stays private. But that needs some knowledge and that's certainly not for everybody.
I am fairly certain that very few people go and read the entire supposedly client-side JS code served by that site. It takes a very tiny bit of obfuscated JS to send 256 bit somewhere else.
[edit]: and when you generate a key pair that's going to potentially store multi kUSD in value, trusting anything that happens in a browser is - call me old fashioned - suicidal.