-
Notifications
You must be signed in to change notification settings - Fork 925
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
chore: consolidate tx confirmation panels #21188
chore: consolidate tx confirmation panels #21188
Conversation
A Storybook has been deployed to preview UI for the latest push |
699a581
to
8b81915
Compare
A Storybook has been deployed to preview UI for the latest push |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
8b81915
to
9fb8142
Compare
A Storybook has been deployed to preview UI for the latest push |
9fb8142
to
11d8243
Compare
A Storybook has been deployed to preview UI for the latest push |
11d8243
to
4f0d2a4
Compare
A Storybook has been deployed to preview UI for the latest push |
4f0d2a4
to
9fcb014
Compare
[puLL-Merge] - brave/brave-core@21188 DescriptionThe PR refactors the ChangesChanges
Security Hotspots
|
A Storybook has been deployed to preview UI for the latest push |
9fcb014
to
a332123
Compare
A Storybook has been deployed to preview UI for the latest push |
* chore: consolidate tx confirmation panels * chore: combine transaction-info components * chore: combine transaction-detail components * fix: allow rows to wrap content * fix: allow tx confirm to-from row to wrap * chore: remove max-width prop from column component
Verification passed on
|
…2.x) (#21625) * chore: consolidate tx confirmation panels (#21188) * chore: consolidate tx confirmation panels * chore: combine transaction-info components * chore: combine transaction-detail components * fix: allow rows to wrap content * fix: allow tx confirm to-from row to wrap * chore: remove max-width prop from column component * fix: display error when unable to fetch transaction fee
Resolves brave/brave-browser#34661
Submitter Checklist:
QA/Yes
orQA/No
;release-notes/include
orrelease-notes/exclude
;OS/...
) to the associated issuenpm run test -- brave_browser_tests
,npm run test -- brave_unit_tests
wikinpm run lint
,npm run presubmit
wiki,npm run gn_check
,npm run tslint
git rebase master
(if needed)Reviewer Checklist:
gn
After-merge Checklist:
changes has landed on
Test Plan:
Transaction confirmation panel should continue to display all pertinent information for all transaction types without major regression