Skip to content

Commit 0e3b7a8

Browse files
committed
Also show pagination interface for interpreted results
1 parent 65aa692 commit 0e3b7a8

File tree

2 files changed

+3
-5
lines changed

2 files changed

+3
-5
lines changed

extensions/ql-vscode/src/interface.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -407,7 +407,7 @@ export class InterfaceManager extends DisposableObject {
407407
database: this._displayedQuery.database,
408408
metadata: this._displayedQuery.query.metadata,
409409
pageNumber,
410-
totalPages: Math.ceil(this._interpretation.sarif.runs[0].results.length / INTERPRETED_RESULTS_PAGE_SIZE),
410+
numPages: Math.ceil(this._interpretation.sarif.runs[0].results.length / INTERPRETED_RESULTS_PAGE_SIZE),
411411
});
412412
}
413413

extensions/ql-vscode/src/view/result-tables.tsx

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -112,12 +112,10 @@ export class ResultTables
112112

113113
/**
114114
* Holds if we actually should show pagination interface right now. This is
115-
* still false for the time being when we're viewing alerts.
115+
* true as long as the result sets we're given are marked to permit it.
116116
*/
117117
paginationEnabled(): boolean {
118-
return this.paginationAllowed() &&
119-
this.props.parsedResultSets.selectedTable !== ALERTS_TABLE_NAME &&
120-
this.state.selectedTable !== ALERTS_TABLE_NAME;
118+
return this.paginationAllowed()
121119
}
122120

123121
constructor(props: ResultTablesProps) {

0 commit comments

Comments
 (0)