-
Notifications
You must be signed in to change notification settings - Fork 5.5k
feat: show helptext and error when both are available #37297
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
Conversation
|
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. |
📊 Page Load Benchmark ResultsCurrent Commit: 📄 Localhost MetaMask Test DappSamples: 100 Summary
📈 Detailed Results
Results generated automatically by MetaMask CI |
Builds ready [03f2906]
UI Startup Metrics (1242 ± 79 ms)
Bundle size diffs [🚨 Warning! Bundle size has increased!]
|
📊 Page Load Benchmark ResultsCurrent Commit: 📄 Localhost MetaMask Test DappSamples: 100 Summary
📈 Detailed Results
Results generated automatically by MetaMask CI |
Builds ready [b182ef4]
UI Startup Metrics (1299 ± 90 ms)
Bundle size diffs [🚨 Warning! Bundle size has increased!]
|
📊 Page Load Benchmark ResultsCurrent Commit: 📄 Localhost MetaMask Test DappSamples: 100 Summary
📈 Detailed Results
Results generated automatically by MetaMask CI |
Builds ready [3fb2405]
UI Startup Metrics (1274 ± 94 ms)
|
📊 Page Load Benchmark ResultsCurrent Commit: 📄 Localhost MetaMask Test DappSamples: 100 Summary
📈 Detailed Results
Results generated automatically by MetaMask CI |
Builds ready [b2dc0c2]
UI Startup Metrics (1284 ± 81 ms)
Bundle size diffs [🚨 Warning! Bundle size has increased!]
|
Description
On
Claims form > Impacted transaction hash, if there is an error on the input, we are hiding the help text together with theNeed help finding it?link. This PR keeps the link showing together with the error so users can still click it.https://consensyssoftware.atlassian.net/browse/SUBS-675
Changelog
CHANGELOG entry: Show Need help link together with error on Claims form Transaction hash field
Related issues
Fixes:
Manual testing steps
Submit a ClaimScreenshots/Recordings
Before
Help text was hidden when there is error

After
Show need help together with the error

Pre-merge author checklist
Pre-merge reviewer checklist
Note
Keep the impacted transaction hash help link visible even when showing an error; minor help text styling tweak on chain ID field.
ui/pages/settings/transaction-shield-tab/submit-claim-form/submit-claim-form.tsx):TextButton.helpTextProps.Written by Cursor Bugbot for commit b2dc0c2. This will update automatically on new commits. Configure here.