The suggestion that I've been given by many people has been Trezor, pretty exclusively Trezor. I've been told that ledger is just a cheaper, less secure version of a Trezor; if you are looking for a storage solution similar to a ledger, but better, than simply opt for the Trezor. Look at their security model, and more information about them on their website.
https://trezor.io/They support tons of coins and offer more protection than any other hardware wallet that I know of.
I personally prefer Trezor over Ledger as well, but from what I've gathered so far both seem to be en par in terms of security.
They both have a good track record in fixing vulnerabilities in a timely manner and as far as I'm aware of neither has had blatant design flaws in terms of security. For what it's worth Ledger Nano S supports longer passphrases than both Trezor versions (100 ASCII characters vs 50 ASCII characters) however this only becomes relevant once you accidentally make your seed phrase available to the public, in which case you should move your coins anyway.
TLDR; both Trezor and Ledger are fine, as long as you purchase them from the producers directly:
https://trezor.io/https://www.ledger.com/products/ledger-nano-sThe rest is up to personal preference in terms of coins supported and usability. AFAIK you can only run a limited number of wallet apps on Ledger at a time while Trezor supports all coins it supports straight out-of-the-box, so keep that one in mind.
On a sidenote, there's also KeepKey which seems to be fairly trusted but rarely used. Other than that there have been a couple of new hardware wallets on the block lately, with no track records and little to show for except for flashy designs and product descriptions full of cryptobabble. I'd stay as far away from those as possible.
