Skip to content
This repository has been archived by the owner on Oct 26, 2021. It is now read-only.

New UniqueTokens card design and layout! #488

Open
wants to merge 6 commits into
base: latest
Choose a base branch
from
Open

Conversation

mirshko
Copy link

@mirshko mirshko commented Jun 1, 2019

PR Checklist (check all)

  • Updated CHANGELOG.md to describe the included fixes and changes made
  • Included issue number on the description of the PR
  • Commented on the relevant issue thread about this PR

Issue number

Changelog

@mirshko
Copy link
Author

mirshko commented Jun 1, 2019

Screenshot 2019-06-01 22 48 07

@mirshko mirshko changed the title New assets screen New UniqueTokens card design and layout! Jun 1, 2019
@mirshko
Copy link
Author

mirshko commented Jun 1, 2019

Its also responsive using css multicolumn!

@ricburton
Copy link
Member

ricburton commented Jun 1, 2019

🙌 Preview this pull request.

Built with commit f472dbf

https://deploy-preview-488--balance-manager.netlify.com

Copy link
Contributor

@jackjdleslie jackjdleslie left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! No functional changes so will be fine to merge

mirshko and others added 2 commits June 2, 2019 12:42
will work on masonry or all same size cards soon
Uses OpenSea placeholder as NFT image, and uses contract name and NFT id for as NFT name
@jackjdleslie jackjdleslie added this to the Manager revamp milestone Jun 2, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants