show asset name instead of symbol for setApprovalForAll calls on NFT contracts where possible (#15296)

ui/pages/confirm-approve/confirm-approve-content/confirm-approve-content.component.js

@@ -457,6 +457,7 @@ export default class ConfirmApproveContent extends Component {
       chainId,
       assetStandard,
       tokenSymbol,
+      isSetApproveForAll,
     } = this.props;
     const { t } = this.context;
     let titleTokenDescription = t('token');
@@ -483,7 +484,10 @@ export default class ConfirmApproveContent extends Component {
       titleTokenDescription = unknownTokenLink;
     }
 
-    if (assetStandard === ERC20 || (tokenSymbol && !tokenId)) {
+    if (
+      assetStandard === ERC20 ||
+      (tokenSymbol && !tokenId && !isSetApproveForAll)
+    ) {
       titleTokenDescription = tokenSymbol;
     } else if (
       assetStandard === ERC721 ||
@@ -492,9 +496,9 @@ export default class ConfirmApproveContent extends Component {
       (assetName && tokenId) ||
       (tokenSymbol && tokenId)
     ) {
-      const tokenIdWrapped = tokenId ? ` (#${tokenId})` : null;
+      const tokenIdWrapped = tokenId ? ` (#${tokenId})` : '';
       if (assetName || tokenSymbol) {
-        titleTokenDescription = `${assetName ?? tokenSymbol} ${tokenIdWrapped}`;
+        titleTokenDescription = `${assetName ?? tokenSymbol}${tokenIdWrapped}`;
       } else {
         const unknownNFTBlockExplorerLink = getTokenTrackerLink(
           tokenAddress,