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

Product Feedback on new designs #2471

Open
kattylucy opened this issue Oct 4, 2024 · 6 comments
Open

Product Feedback on new designs #2471

kattylucy opened this issue Oct 4, 2024 · 6 comments
Assignees

Comments

@kattylucy
Copy link
Contributor

kattylucy commented Oct 4, 2024

I created this ticket for product to use as the placeholder and add the feedback during testing.

What are we testing?

Landing page (/pools)
Pool overview page (/pools/:poolId)
Reports page (pools/reporting/)
Data page (pools/data)

It relates to all the work in the following tickets

#2392
#2391
#2390
#2427
#2459
#2460

@kattylucy kattylucy self-assigned this Oct 4, 2024
@0x4Graham
Copy link

Hey!

Looks like updating the Target APY isn't working:
image
image

Never updated
image

@0x4Graham
Copy link

image Very very difficult to click on this

@sonam-jo
Copy link

sonam-jo commented Oct 4, 2024

For funds that have only 1 tranche, we should just have the name of the fund under Token, and Token price in the chart -- no junior -- same as the UI today
Screenshot 2024-10-04 at 13 14 15
Screenshot 2024-10-04 at 13 13 16

@sonam-jo
Copy link

sonam-jo commented Oct 4, 2024

Axis for APY chart looks off -- it neither is the same for all time windows, nor does it adjust to show the graph properly with different time windows

Screenshot 2024-10-04 at 13 16 14
Screenshot 2024-10-04 at 13 16 23
Screenshot 2024-10-04 at 13 16 33

@sonam-jo
Copy link

sonam-jo commented Oct 4, 2024

The only item missing from this Overview box is Expense ratio.

  1. Expense ratio is something we calculate, and should appear in every fund we create as long as we have the fees numbers and the NAV.
  2. There is a lot of blank space in the overview box. I would recommend increasing the spacing across the rows in line with Figma, and perhaps reducing the side of the chart so there is less white space on the page.
  3. There seems to be a difference in the font size for the titles (e.g. Asset type) and the respective fields (e.g. Private credit)

Screenshot 2024-10-04 at 13 19 33

@sonam-jo
Copy link

sonam-jo commented Oct 4, 2024

Transaction history table is not the same as FIgma

Screenshot 2024-10-04 at 13 24 36
Screenshot 2024-10-04 at 13 24 47

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

3 participants