Mark Stacey
429451de23
Extract "create vault" form to separate component ( #13461 )
...
The form used for creating a vault on the "Import" page of onboarding
and on the "Restore vault" page is nearly identical, yet the
implementation is totally separate. It has now been extracted to a
separate component, consolidating the two implementations.
There is a "terms of use" checkbox on the import page that isn't on the
restore vault page, so that part has been made optional. The "submit"
button text differs between the two uses as well, so that is
customizable.
There are slight styling differences between the old and new versions
of this form. The fonts and spacing are all using our new standard
design system guidelines, and we're using our standard checkbox now as
well. The spacing and font sizes were chosen somewhat arbitrarily by me
to resemble the old styles, so please feel free to suggest changes if
you think they can be improved upon.
There are some slight copy changes to the "Restore vault" page as well;
the placeholder text and the label for the "Secret Recovery Phrase"
field now matches the "Import" page copy.
3 years ago
Jyoti Puri
760ed3457d
Removing EIP_1559_V2 feature flag ( #13481 )
3 years ago
MG
3301fd8121
Feat/hide 0 balances ( #13306 )
...
* Added token list functional component
* Added list - img not showing up
* Changed render list
* Removed unused code
* Clean up
* Lint
* Linted
* Add newline
* Filter out isERC721
* Added token list
* Cleaned up style
* Fixed typography
* Fixed lint
* Fixed spacing measure
* Lint cleanup
3 years ago
ryanml
01844aef53
Handling array type values for image in Identicon component ( #13484 )
...
* Handling array type values for image in Identicon component
* Adding additional type check
3 years ago
ryanml
5e07c0768b
Handling array type values for image in Identicon component ( #13484 )
...
* Handling array type values for image in Identicon component
* Adding additional type check
3 years ago
Jyoti Puri
cc1861a34e
Adding metrics events for EIP-1559 V2 flow ( #13329 )
3 years ago
Ariella Vu
a5b114c4f9
Use ActionableMessage for ConfirmAddSuggestionToken warnings ( #13402 )
...
* ConfirmAddSuggestedToken: update duplicate logic
* ConfirmAddSuggestedToken: update duplicate fn names
* ConfirmAddSuggestedToken: add ActionableMessage
- fixes #13347
* base-styles.scss: deprecate .warning
* ConfirmAddSuggestedTokens: update @param desc
Co-authored-by: Alex Donesky <adonesky@gmail.com>
* ConfirmAddSuggestedTokens: update @param desc
Co-authored-by: Alex Donesky <adonesky@gmail.com>
* ConfirmAddSuggestedTokens: update @param desc
Co-authored-by: Alex Donesky <adonesky@gmail.com>
* ConfirmAddSuggestedTokens: update @param desc
Co-authored-by: Alex Donesky <adonesky@gmail.com>
* ConfirmAddSuggestedToken: clean JSDoc comments
- following GitHub suggestion commits
* ConfirmAddSuggestedToken: warning Learn More btn
* add zendesk-url constants
- use for ConfirmAddSuggestedToken & ImportToken pages
* storybook: add ConfirmAddSuggestedToken controls
* stories: add ConfirmAddSuggestedTokens states
* stories: rm unused args ConfirmAddSuggestedToken
* rn ZENDESK_URL -> ZENDESK_URLS
* AddSuggestedToken: check case-insensitive symbols
Co-authored-by: Alex Donesky <adonesky@gmail.com>
3 years ago
Ariella Vu
3a75f68a15
#10168 - Fix to Can't open settings if browser zoom level > 100% part 2 ( #13460 )
...
* AccountMenu: fix #10168 w/ position: absolute
- use right: 1rem to match .app-header padding
* scss: Use px instead of rem for positioning
Co-authored-by: George Marshall <george.marshall@consensys.net>
Co-authored-by: George Marshall <george.marshall@consensys.net>
3 years ago
Hassan Malik
d02facf17e
changed icon to button for connected-sites-list ( #13476 )
...
* changed icon to button for connected-sites-list
* made CSS changes
3 years ago
David Walsh
c1b5e18a01
Avoid repeating CSS color vars ( #13464 )
3 years ago
Alex Donesky
dc217dd536
Yet more NFT UX cleanups ( #13435 )
...
* yet more nft ux cleanups
3 years ago
Shane
09ac38977e
Add TextArea to safe components list ( #13468 )
...
These changes were split off from the snaps branch [here](https://github.com/MetaMask/metamask-extension/pull/11837#discussion_r792944264 ).
3 years ago
Shane
516de24cbe
Fixed multiple-notifications css ( #13469 )
3 years ago
Jyoti Puri
11c61a3ead
EIP-1559 V2 UI/UX improvements ( #13419 )
3 years ago
Ariella Vu
869d7a21bf
Revert "#10168 - Fix to Can't open settings if browser zoom level > 100% ( #13281 )" ( #13459 )
...
This reverts commit 05f9ceda77
.
3 years ago
Shane
bac20099e5
Removed memo status from snap-settings-card ( #13458 )
3 years ago
Hassan Malik
2825b74a55
added extension changes from snaps branch ( #13433 )
3 years ago
yanjade
3eb99babf2
batch fix misspelling ( #13449 )
3 years ago
David Walsh
851fce9a3b
Enable buying MATIC on Polygon, BNB on BSC, AVAX on Avalanche, FTM on Fantom, CELO on Celo, and respective stablecoins on Transak ( #13398 )
3 years ago
thickforest
fd46f97e35
Fix className misspelling in `awaiting-swap.js `
3 years ago
Alex Donesky
be65eb7339
More nft ux fixes ( #13388 )
...
* a batch of nft ux fixes
3 years ago
Jyoti Puri
a0e97f4681
EIP-1559 v2 - small fixes ( #13392 )
3 years ago
filipsekulic
fcd5f89e2f
Move component into another folder ( #13380 )
3 years ago
Jyoti Puri
eb39c51ede
Using new graph images for edit gas fee popover ( #13382 )
3 years ago
VSaric
7b75b9e638
Fix failing test:unit pipeline ( #13387 )
...
* Fix failing test:unit pipeline
* Correct check of gasLimit and properGasLimit in edit-gas-display
Co-authored-by: Dan Miller <danjm.com@gmail.com>
3 years ago
Ariella Vu
667350d15a
Convert ConfirmAddSuggestedToken to a functional component + cleanup ( #13377 )
...
* FC: ConfirmAddSuggestedToken
- change class to functional component
* clean: reorganize functions (no logic change)
* FC: ConfirmAddSuggestedToken pt. 2
- preserve trackEvent logic -> metricsEvent
* clean: rm _ prefix from fn name
- why? now, only 3 other functions remain prefixed w/ '_'
* clean: update useEffect hook
* clean: alphabetize props
* clean: functions in ConfirmAddSuggestedToken
* clean: use arrow fn for FC
3 years ago
MG
8756ad2e78
Feat: Label hardware wallets ( #13339 )
...
* Added keyring label
* Fixed labels name
* Clean up
* Remove blank spaces
* Linted
* Fixed lint
* Removed unused hooks
* Fixed test
* Addressed PR comments
* Added KEYRING_NAMES obj
* Lint
* Removed empty space
3 years ago
Jon Ator
b1f5f78d90
Maintain leading whitespace in sig req window ( #13340 )
3 years ago
Jyoti Puri
119e5497ff
UI fixes in EIP-1559 V2 screens ( #13367 )
3 years ago
Jyoti Puri
d25f0e5817
Fix for checkbox on advance gas fee modal ( #13366 )
3 years ago
VSaric
5579061cfa
Proper calculation of the gas limit ( #12784 )
3 years ago
Alex Donesky
db0230f23a
NFT UX cleanup grab-bag ( #13352 )
...
* NFT UX cleanup
3 years ago
filipsekulic
b2a9b72c04
Refactor currency input component and ensure currency toggle persists between settings ( #12813 )
3 years ago
dragana8
a67a5efca3
Incorrect Rounding Down of Balance #10167 ( #13337 )
3 years ago
Ariella Vu
683aa22b56
Update global link hover color from orange to blue (--Blue-400) ( #13344 )
...
* base-styles.scss: update a:hover to --Blue-400
- deprecates #df6b0e
* preserve a.warning:hover orange color w/ #ceafa22
* Revert "preserve a.warning:hover orange color w/ #ceafa22"
This reverts commit d0efb923808bbc43e7f955fd8ce3e95f6a124c1f.
* create-account: rm .warning style from help link
- supports #ceafa2
3 years ago
Alex Donesky
f087e501a1
Add modal with directions to re-add token as NFT ( #13291 )
...
* Add modal pop for NFTs previously added as tokens - with directions to re-add as NFTs
3 years ago
Jyoti Puri
5b92dc4cf0
Adding e2e test for edit gas fee modal ( #13303 )
3 years ago
Jyoti Puri
5b5b2cd7df
Fixing warnings in unit test cases ( #13349 )
3 years ago
6201
af848dec9b
Create "inline" type for Button component ( #13304 )
3 years ago
Alex Donesky
f7849a0b7c
Add error that redirects users to Import NFT page when they attempt to add an NFT on the Import Token page ( #13271 )
...
* Add error that redirects users to Import NFT page when they attempt to add an NFT on the Import Token page
3 years ago
Jyoti Puri
9a3c917a48
Adding support for EIP-1559 in E2E tests ( #13282 )
3 years ago
Elliot Winkler
66ceeefae3
Fix UI issues at top of confirm page container ( #13314 )
...
This commit fixes a few issues with ConfirmPageContainerSummary (which
holds the contract being used or action being performed and the money
being sent):
* Remove fixed height so that the secondary currency doesn't get cut off
or spill over
* Add missing padding
* Fix font size of primary and secondary currencies
* Add top border when there is a "address not in your address book"
alert at the top
Co-authored-by: Ariella <ariellavu@gmail.com>
3 years ago
Niranjana Binoy
644e08eaea
V10.9.0: Disconnect all accounts flow ( #13309 )
3 years ago
Jyoti Puri
bd33512c2a
Fix issue with dapp suggested option being visible in edit gas fee popover even if dapp has no gas suggestion ( #13316 )
3 years ago
Benjamin Bourgeois
d12097ac1c
UpdateNickname story : convert knobs and actions to controls/args ( #13189 )
...
Co-authored-by: George Marshall <georgewrmarshall@gmail.com>
3 years ago
Matthew Epps
a8ec876031
Fix token ordering: prioritize top before owned ( #13270 )
3 years ago
6201
371da2eb83
Box story: convert knobs to controls ( #13274 )
...
Co-authored-by: caogu <sunlon1987@gmail.com>
3 years ago
VSaric
cb815a91ad
SendHeader, ContactListTab and AdvancedTab story: converted knobs and actions to controls / args ( #13230 )
...
* SendHeader, ContactListTab and AdvancedTab story: converted knobs and actions to controls / args
* Added the other function type props to action controls
* Requested review changes
3 years ago
Elliot Winkler
8beaae3361
Fix UI issues at top of confirm page container ( #13314 )
...
This commit fixes a few issues with ConfirmPageContainerSummary (which
holds the contract being used or action being performed and the money
being sent):
* Remove fixed height so that the secondary currency doesn't get cut off
or spill over
* Add missing padding
* Fix font size of primary and secondary currencies
* Add top border when there is a "address not in your address book"
alert at the top
Co-authored-by: Ariella <ariellavu@gmail.com>
3 years ago
Niranjana Binoy
58ff673bcd
V10.9.0: Disconnect all accounts flow ( #13309 )
3 years ago