Skip to content
This repository has been archived by the owner on May 13, 2024. It is now read-only.

Commit

Permalink
Merge pull request #778 from derrabauke/fix-analysis-record-count-ind…
Browse files Browse the repository at this point in the history
…ication

fix(analysis): show number of displayed reports
  • Loading branch information
derrabauke authored Jan 24, 2023
2 parents cb50173 + e405251 commit 6d0cfa1
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 5 deletions.
4 changes: 4 additions & 0 deletions app/analysis/index/controller.js
Original file line number Diff line number Diff line change
Expand Up @@ -137,6 +137,8 @@ export default class AnalysisController extends Controller.extend(
@tracked _shouldLoadMore = false;
@tracked _canLoadMore = true;
@tracked _lastPage = 0;
@tracked totalTime = moment.duration();
@tracked totalItems = A();
@tracked selectedReportIds;

@tracked user;
Expand Down Expand Up @@ -207,6 +209,8 @@ export default class AnalysisController extends Controller.extend(
this._shouldLoadMore = false;
this._dataCache = A();
this.selectedReportIds = A();
this.totalTime = moment.duration();
this.totalItems = A();

this.data.perform();
}
Expand Down
7 changes: 2 additions & 5 deletions app/analysis/index/template.hbs
Original file line number Diff line number Diff line change
Expand Up @@ -397,7 +397,7 @@
false
}}</strong></td>
<td colspan="6" class="text-right"><em>Displaying
{{this.reports.length}}
{{reports.length}}
of
{{this.totalItems}}
reports</em></td>
Expand Down Expand Up @@ -431,10 +431,7 @@
disabled={{this.exportLimitExceeded}}
title={{if this.exportLimitExceeded this.exportLimitMessage}}
>
<FaIcon
@icon="download"
@prefix="fas"
/>&nbsp;{{link.label}}
<FaIcon @icon="download" @prefix="fas" />&nbsp;{{link.label}}
</button>
{{/each}}
</div>
Expand Down

0 comments on commit 6d0cfa1

Please sign in to comment.