Remove "Base58 (P2SH-SegWit)" address generation in UI and do not show them in minting tab

[MatthewLM]

In the receive tab, there are options for "Base58 (Legacy)", "Base58 (P2SH-SegWit)" and "Bech32 (SegWit)". The P2SH_SEGWIT outputs are not used by the client minting code. Therefore they should be removed as an option in src/qt/receivecoinsdialog.cpp. These outputs should also not be included in the minting tab as they cannot be minted by the client code.

Also consideration should be had about the tooltips. The base58 P2PKH (LEGACY) option currently has the message "Not recommended due to higher fees and less protection against typos.".