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

fix tick format of Payments charts #4385

Conversation

mierin12
Copy link
Contributor

@mierin12 mierin12 commented Dec 3, 2024

Hello,
This is a proposition to fix and simplify the tick format of Payments charts (use ThousandAmoundFormat for all payments charts).
Y axis tick format are defined as ThousandsNumberFormat in PaymentsPerMonthChartBuilder, PaymentsPerYearChartBuilder, PaymentsPerQuarterChartBuilder but also in the higher level PaymentsChartTab where it is either ThousandsNumberFormat either AmountNumberFormat.

It seems that the PaymentsChartTab overwrites the other ones.
But it also does not update the format on tab switching : if you first open a chart with low values, such as Fees, then AmountNumberFormat is set and then, even if you switch to Savings or a >1k chart, it will stays AmountNumberFormat :
Capture d’écran 2024-12-03 220516

But is AmountNumberFormat really needed ? Since 3cb9ee1, the ThousandsNumberFormat is not adding "k" to <1000 values. So fine even for small value.
AmountNumberFormat is more for tooltip I believe.

An even more simpler way than this PR would be to set the tick format as Thousands once in PaymentsChartTab and remove it in the four Payments charts (per month/quarter/year, accumulated). It should work too I guess.

@buchen buchen merged commit b35886d into portfolio-performance:master Dec 7, 2024
2 checks passed
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

Successfully merging this pull request may close these issues.

2 participants