-
Notifications
You must be signed in to change notification settings - Fork 20
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
Combine all APYs into a single value #142
Comments
End result will look like something like this: Tooltip: Please address these two comments and once ready, we can move this ticket to unblock |
tagging @michwill to weigh in here. |
Discussed and:
proposed tooltip looks fine:
|
I think yes, we switch to APR here |
Let's do APR
That's fine on desktop view. On mobile view, since there's no hover, can probably do line |
A suggestion from a prolific user was to combine all APYs into a single number, such that a user simply needs to do a hover over the number to get a full breakdown of the yield sources.
An example from Balancer:
This also allows us to reduce the number of columns by 1.
The fe should then show a min and max value, with a breakdown of of the min being base + unboosted incentives, and max being base + boosted incentives.
The text was updated successfully, but these errors were encountered: