Burcu Noyan comments

Results 9 comments of


                                            Burcu Noyan

User should be able to sort assets

@dnaidionov should this assets view be a new route (`/assets`)?

Extra whitespace around isolated asset makes animation feel jumpy

Yes that was accidental. #48 should fix it.

Extra whitespace around isolated asset makes animation feel jumpy

The extra white space is gone but need to how smooth the animation is after #67 is deployed to staging because the new design has a header on the portfolio...

Navigation history within the site should be represented as breadcrumbs

What's left: - [ ] Add counts (see in the design) - [ ] Linking to wallet (currently hardcoded) - [ ] Linking to asset (currently only using "transitionTo" address...

Asset crumb in breadcrumbs doesn't work

This is temporarily fixed in #67 by using "transitionTo" property instead of "to address". For detailed fix info see #62

Asset crumb in breadcrumbs doesn't work

Oh wait. This doesn't work anymore because "transitionTo" property converts address to lowercase ...

Home screen - Wallets view

#67 addresses general UI. If there's no wallet, instead of empty page, there's a message that says "Please add a wallet to get started." We can improve what's displayed here...

Asset card - isolated

To Do: - [ ] Belongs To box: Query wallet and display wallet name + logo - [ ] Add "Transfer Funds", "Trade Currency", and more options buttons??? - [...

Asset card - isolated

For embedded asset card: - [ ] Exchange rate & tags should display on hover