fix: fixed wallet view bugs for connections and permissions screen

[NidhiKJha]

This PR is to fix the UI bugs in the wallet, permissions and connections screen to keep it in sync with the Updated designs

Description

Following changes are updated:

1. Description of the product tour for global menu [Note: This product tour will be removed]
2. Remove Connected dApp shown on Account List Item
3. Remove Connect account action in Select Account Modal

Related issues

Fixes: #23858

Manual testing steps

This PR solves Part of the linked issue, rest of them are already resolved via different other PRs, Linking them in the list:

1. Description of the product tour for global menu - Modified in this PR
2. Connected sites and All Permissions in Global menu - 23789
3. Remove Connected dApp shown on Account List Item - Modified in this PR
4. Connected Account Toast should only be displayed on Wallet screen - 23733
5. Remove Connect account action in Select Account Modal - Modified in this PR
6. Network switching within Extension - This isssue doesn't exist anymore.
7. Button in Network switch Modal - Was related to Design System, fixed.

Screenshots/Recordings

Before

Please refer to the issue

After

1. This Product tour will be removed in a separate PR
2. No dapp logo on account list item even when it's connected
   
3. No connect button in account list item menu

Pre-merge author checklist

• I’ve followed MetaMask Coding Standards.
• I've completed the PR template to the best of my ability
• I’ve included tests if applicable
• I’ve documented my code using JSDoc format if applicable
• I’ve applied the right labels on the PR (see labeling guidelines). Not required for external contributors.

Pre-merge reviewer checklist

• I've manually tested the PR (e.g. pull and build branch, run the app, test code being changed).
• I confirm that this PR addresses all acceptance criteria described in the ticket it closes and includes the necessary testing evidence such as recordings and or screenshots.

[github-actions]

CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 67.52%. Comparing base (9c78339) to head (27bb99a).

Additional details and impacted files

@@             Coverage Diff             @@
##           develop   #24023      +/-   ##
===========================================
- Coverage    67.52%   67.52%   -0.01%     
===========================================
  Files         1244     1244              
  Lines        48807    48798       -9     
  Branches     12747    12742       -5     
===========================================
- Hits         32955    32946       -9     
  Misses       15852    15852              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[metamaskbot]

Builds ready [27bb99a]

• builds: chrome, firefox
• builds (beta): chrome
• builds (flask): chrome, firefox
• builds (MMI): chrome, firefox
• builds (test): chrome, firefox
• builds (test-flask): chrome, firefox
• build viz: Build System
• mv3: Background Module Init Stats
• mv3: UI Init Stats
• mv3: Module Load Stats
• mv3: Bundle Size Stats
• mv2: E2e Actions Stats
• code coverage: Report
• storybook: Storybook
• typescript migration: Dashboard
• all artifacts
• bundle viz:

  • background: 0, 1, 2, 3, 4, 5, 6
  • common: 0, 1, 2, 3, 4, 5, 6, 7, 8
  • content-script: 0
  • ui: 0, 1, 10, 2, 3, 4, 5, 6, 7, 8, 9

Page Load Metrics (1124 ± 609 ms)

Platform	Page	Metric	Min (ms)	Max (ms)	Average (ms)	StandardDeviation (ms)	MarginOfError (ms)
Chrome	Home	firstPaint	60	218	126	43	21
		domContentLoaded	10	65	30	16	8
		load	50	2946	1124	1269	609
		domInteractive	10	65	30	16	8

Bundle size diffs [🚀 Bundle size reduced!]

• background: 0 Bytes (0.00%)
• ui: -403 Bytes (-0.01%)
• common: -2 Bytes (-0.00%)