chore: sync remaining copy and tooltips with mobile figma#41233
Merged
Conversation
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. |
Contributor
✨ Files requiring CODEOWNER review ✨✅ @MetaMask/confirmations (6 files, +96 -36)
👨🔧 @MetaMask/metamask-earn (6 files, +42 -21)
|
matthewwalsh0
previously approved these changes
Mar 26, 2026
…s-and-toast-message-across-single-convert-experience-1
Contributor
Builds ready [eea65e8]
⚡ Performance Benchmarks
🌐 Dapp Page Load BenchmarksCurrent Commit: 📄 Localhost MetaMask Test DappSamples: 100 Summary
📈 Detailed Results
Bundle size diffs [🚨 Warning! Bundle size has increased!]
|
matthewwalsh0
previously approved these changes
Mar 26, 2026
…-message-across-single-convert-experience-1' of https://github.com/MetaMask/metamask-extension into MUSD-465-extension-review-and-update-tooltips-and-toast-message-across-single-convert-experience-1
pedronfigueiredo
approved these changes
Mar 26, 2026
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.
Jwhiles
approved these changes
Mar 26, 2026
|
Contributor
Builds ready [ea74348]
⚡ Performance Benchmarks
🌐 Dapp Page Load BenchmarksCurrent Commit: 📄 Localhost MetaMask Test DappSamples: 100 Summary
📈 Detailed Results
Bundle size diffs [🚨 Warning! Bundle size has increased!]
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.




Description
Polish mUSD conversion UX: education headline/body use pre-line for intended line breaks; confirmation header info popover adds Relay attribution, aria-label, and a wider popover (maxWidth 250); claimable bonus row tooltip uses new copy placeholders and aria-label. Home Get/Buy mUSD CTA reuses musdBoostTitle / musdBoostDescription and matches asset CTA styling tweaks. Removed unused musdEarnBonusPercentage from locales (verify-locales). en_GB kept identical to en.
Changelog
CHANGELOG entry: Updated mUSD conversion education and confirmation copy: clearer bonus explanation layout,
Powered by Relayon the header info popover, wider info popover (maxWidth250px), accessible names on bonus tooltips, aligned home mUSD CTA copy/typography with the asset overview CTA, and removed unusedmusdEarnBonusPercentagelocale entries after reusing boost strings.Related issues
Fixes: https://consensyssoftware.atlassian.net/browse/MUSD-465
Manual testing steps
Feature: mUSD conversion copy and tooltips
Scenario: Education headline respects line breaks
Given the mUSD conversion education screen is shown
Then the main heading should show the bonus headline with a line break between "ON" and "STABLECOINS"
Scenario: Header bonus popover shows attribution and is accessible
Given an mUSD conversion confirmation is shown
When the user focuses or uses the header bonus info control
Then the control should have an accessible name for bonus details
When the user opens the bonus info popover
Then the popover should show "Powered by Relay" and use a wider max width
Scenario: Claimable bonus tooltip is accessible
Given the claimable bonus row is visible on the conversion confirmation
Then the info control should have an accessible name for claimable bonus details
Scenario: Home mUSD CTA matches boost messaging
Given the home mUSD Get/Buy CTA is shown
Then the primary and secondary lines should show the boost title and description with the configured APY
Screenshots/Recordings
Before
After
Pre-merge author checklist
Pre-merge reviewer checklist
Note
Low Risk
Low risk: mostly copy/layout tweaks, small style adjustments, and added
aria-labels for tooltip triggers; no transaction or security logic changes.Overview
Polishes the mUSD conversion UX by updating localized strings and restructuring bonus/claim tooltips: the bonus explanation copy is reflowed with intended line breaks,
Powered by Relayis added as separate attribution, and claimable-bonus tooltip placeholders are simplified (dropping the APY interpolation) while removing the unusedmusdEarnBonusPercentagekey.Improves accessibility and UI consistency by adding context-specific
ariaLabelsupport toInfoPopoverTooltipand wiring new accessible-name strings for the conversion and claimable-bonus info controls, widening the tooltip popover max width (240→250), and aligning home/asset mUSD CTA typography and icon styling. Tests are updated accordingly.Written by Cursor Bugbot for commit ea74348. This will update automatically on new commits. Configure here.