Skip to content
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

Design for Shapeshift transactions in New UI #3214

Closed
danjm opened this issue Feb 8, 2018 · 7 comments
Closed

Design for Shapeshift transactions in New UI #3214

danjm opened this issue Feb 8, 2018 · 7 comments
Assignees
Labels
needs-design Needs design support.

Comments

@danjm
Copy link
Contributor

danjm commented Feb 8, 2018

When shapeshift transactions are shown in the transaction list in the new UI, we are using a design that borrows heavily from the shapeshift transactions in the old-ui. We might want to update these at some point.

screen shot 2018-02-08 at 11 01 51 am

@danjm danjm added the needs-design Needs design support. label Feb 8, 2018
@cjeria
Copy link
Contributor

cjeria commented May 15, 2018

Good catch @danjm. I'll put this my shortlist of designs to pick up for new UI release epic.

@cjeria
Copy link
Contributor

cjeria commented May 17, 2018

What other data can we pull from shapeshift transactions?
To addresses
Amount being transferred

@snreynolds
Copy link

I also noticed that the first couple of screens are a bit messed up in the new UI as well. Here is potential fix/update for the screen when users are first connecting to shapeshift.
screen shot 2018-06-21 at 2 01 49 pm

@cjeria
Copy link
Contributor

cjeria commented Jun 21, 2018

Good catch @snreynolds . Here's a screenshot of how it looks right now. Looks like it can easily be fixed (centered on the screen, padding fixed etc). We have specific react components already in place for pop ups such as this screen, so for now, let's stick to our current defined styles.
image

@cjeria
Copy link
Contributor

cjeria commented Jun 21, 2018

created a new issue for this UI bug #4637

@bdresser
Copy link
Contributor

bdresser commented Aug 7, 2018

Not going to have this block removing the old UI, moved out of epic~

@whymarrh
Copy link
Contributor

We've dropped support for Shapeshift transactions, so this is unneeded now

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs-design Needs design support.
Projects
None yet
Development

No branches or pull requests

5 participants