Marmelator/red-ui
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases 1.4k Wiki Activity

RED-4590: Replace else statments with early returns.

Browse Source
This commit is contained in:
George 2023-02-06 14:49:38 +02:00
parent c88da50045
commit 588432a0db
1 changed files with 20 additions and 20 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

40
apps/red-ui/src/app/modules/file-preview/components/file-workload/file-workload.component.ts
View File

@ -410,32 +410,32 @@ export class FileWorkloadComponent extends AutoUnsubscribe implements OnDestroy
private _navigatePages($event: KeyboardEvent) {
const pageIdx = this.displayedPages.indexOf(this.activeViewerPage);
if ($event.key === 'ArrowDown') {
if (pageIdx !== -1) {
// If active page has annotations
if (pageIdx !== this.displayedPages.length - 1) {
this.pdf.navigateTo(this.displayedPages[pageIdx + 1]);
}
} else {
// If active page doesn't have annotations
const nextPage = this._nextPageWithAnnotations();
if (nextPage) {
this.pdf.navigateTo(nextPage);
}
}
} else {
if (pageIdx !== -1) {
// If active page has annotations
if (pageIdx !== 0) {
this.pdf.navigateTo(this.displayedPages[pageIdx - 1]);
}
} else {
if ($event.key !== 'ArrowDown') {
if (pageIdx === -1) {
// If active page doesn't have annotations
const prevPage = this._prevPageWithAnnotations();
if (prevPage) {
this.pdf.navigateTo(prevPage);
}
return;
}
// If active page has annotations
if (pageIdx !== 0) {
this.pdf.navigateTo(this.displayedPages[pageIdx - 1]);
}
return;
}
if (pageIdx === -1) {
// If active page doesn't have annotations
const nextPage = this._nextPageWithAnnotations();
if (nextPage) {
this.pdf.navigateTo(nextPage);
}
return;
}
// If active page has annotations
if (pageIdx !== this.displayedPages.length - 1) {
this.pdf.navigateTo(this.displayedPages[pageIdx + 1]);
}
}