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

Merge Portfolio details into one view #3

Open
greghint opened this issue Nov 16, 2018 · 0 comments
Open

Merge Portfolio details into one view #3

greghint opened this issue Nov 16, 2018 · 0 comments

Comments

@greghint
Copy link
Member

A key part of the demo is to show buying stock increases Loyalty Level, and providing feedback affects sentiment. With the two tabs, it requires the user to buy/sell stock, then toggle over to see any change in Loyalty Level (or to see current sentiment before submitting new feedback).

I suggest you alter the design to move "User Details" to the top in 2 columns above the stock overview.

Specifically:

  • 2 columns for details on top
  • Table for portfolio underneath
  • Add text for Sentiment next to icon (to show what API returns)
  • For the message, put in same spot above portfolio table

image

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

No branches or pull requests

1 participant