Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: update tooltip for swap flow #27261

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

martahj
Copy link
Contributor

@martahj martahj commented Sep 18, 2024

Description

Updates the swap tooltip text and link.

The new text should be:

MetaMask doesn't make money from gas fees. These fees are estimates and can change based on how busy the network is and how complex a transaction is. Learn more here.

[“here” is a hyperlinkl that should link to help center FAQs on gas fees: Gas fees | MetaMask Help Center 🦊♥️ ]

Also updated how a variable was referenced because it was leading to crashes on the legacy swaps flow.

Open in GitHub Codespaces

Manual testing steps

Swap redesign feature flag is here.

  1. Disable smart transactions.

With swap redesign

  1. Make sure the swap redesign feature is enabled in Test env
  2. Start the swap flow
  3. Mouse over the gas fee tooltip
  4. Confirm that it now matches the above behavior

Without swap redesign

  1. Make sure the swap redesign feature is disabled in Test env
  2. Start the swap flow
  3. Proceed to the review screen
  4. Mouse over the gas fee tooltip
  5. Confirm that it now matches the above behavior

Screenshots/Recordings

Before

With swap redesign

Screenshot 2024-09-18 at 8 57 22 AM

Without swap redesign

(this part of the flow is crashing depending on the value of the gasFeeEstimates variable)
Screenshot 2024-09-18 at 2 06 16 PM

After

Screenshot 2024-09-18 at 12 43 40 PM

Pre-merge author checklist

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.

Copy link
Contributor

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.

@metamaskbot metamaskbot added the INVALID-PR-TEMPLATE PR's body doesn't match template label Sep 18, 2024
Copy link

sonarcloud bot commented Sep 18, 2024

@metamaskbot
Copy link
Collaborator

Builds ready [3aec3be]
Page Load Metrics (1681 ± 58 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint20921531631352169
domContentLoaded14801964165710550
load14882063168112258
domInteractive22166433215
Bundle size diffs [🚨 Warning! Bundle size has increased!]
  • background: 0 Bytes (0.00%)
  • ui: -2.04 KiB (-0.03%)
  • common: 459 Bytes (0.01%)

@martahj martahj marked this pull request as ready for review September 18, 2024 20:29
@martahj martahj requested review from a team as code owners September 18, 2024 20:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
INVALID-PR-TEMPLATE PR's body doesn't match template team-swaps
Projects
Status: Needs dev review
Development

Successfully merging this pull request may close these issues.

3 participants