From 86ed715458da712e1010c44c53e85f2661f5548b Mon Sep 17 00:00:00 2001 From: user1823 <92206575+user1823@users.noreply.github.com> Date: Sat, 29 Mar 2025 07:45:41 +0530 Subject: [PATCH] Hide AverageForDaysStudied when studiedPercent = 100 (#3888) Showing both is redundant when studiedPercent is 100 --- ts/routes/graphs/reviews.ts | 12 +++++++----- 1 file changed, 7 insertions(+), 5 deletions(-) diff --git a/ts/routes/graphs/reviews.ts b/ts/routes/graphs/reviews.ts index 76a406b5a..2d590e3fb 100644 --- a/ts/routes/graphs/reviews.ts +++ b/ts/routes/graphs/reviews.ts @@ -396,17 +396,19 @@ export function renderReviews( { label: tr.statisticsTotal(), value: totalString }, - { - label: tr.statisticsAverageForDaysStudied(), - value: averageForDaysStudied, - }, - { label: tr.statisticsAverageOverPeriod(), value: averageForPeriod, }, ]; + if (studiedPercent < 100) { + tableData.push({ + label: tr.statisticsAverageForDaysStudied(), + value: averageForDaysStudied, + }); + } + if (averageAnswerTime) { tableData.push({ label: averageAnswerTimeLabel, value: averageAnswerTime }); }