Select option between P2SH SegWit and Bech32 SegWit is good idea and less confusing for new user, even though it should show info the different both of them.
But i don't see any reason to show legacy address by default since the user simply can use P2SH SegWit which offer backward-compability with more efficient transaction/witness size and helping migration for future technology support such as LN.
There's little reason to use legacy addresses except out of principle (i.e. those who hate segwit). For such people, they can start Bitcoin Core with addresstype=legacy, and hopefully those options will be available from the settings menu too (there's an open PR for that).
True, but i think those who hate SegWit no longer use Bitcoin or already use altcoin which don't use SegWit.