[MNT-23648] Clicking on Load More button no longer causes scroll position to reset to top (#8951)

* [MNT-23648] Resolved issue where clicking on load more button would reset the scroll position to top when copying/moving a folder

* [MNT-23648] Revert fix

* [MNT-23648] Resolved issue where clicking on load more button would reset the scroll position to top when copying/moving a folder

* [MNT-23648] Added test case to cover new functionality

* [MNT-23648] Fixed unit tests
This commit is contained in:
swapnil-verma-gl
2023-10-03 13:21:36 +05:30
committed by GitHub
parent 0b1fc4faa4
commit e638b54fcd
4 changed files with 37 additions and 13 deletions

View File

@@ -550,12 +550,12 @@ export class DocumentListComponent implements OnInit, OnChanges, OnDestroy, Afte
}
}
reload() {
reload(hideLoadingSpinner = false) {
this.resetSelection();
this.reloadWithoutResettingSelection();
this.reloadWithoutResettingSelection(hideLoadingSpinner);
}
reloadWithoutResettingSelection() {
reloadWithoutResettingSelection(hideLoadingSpinner = false) {
if (this.node) {
if (this.data) {
this.data.loadPage(this.node, this._pagination.merge, null);
@@ -565,7 +565,7 @@ export class DocumentListComponent implements OnInit, OnChanges, OnDestroy, Afte
this.syncPagination();
this.onDataReady(this.node);
} else {
this.loadFolder();
this.loadFolder(hideLoadingSpinner);
}
}
@@ -713,8 +713,8 @@ export class DocumentListComponent implements OnInit, OnChanges, OnDestroy, Afte
}
}
loadFolder() {
if (!this._pagination.merge) {
loadFolder(hideLoadingSpinner = false) {
if (!hideLoadingSpinner) {
this.setLoadingState(true);
}
@@ -944,9 +944,8 @@ export class DocumentListComponent implements OnInit, OnChanges, OnDestroy, Afte
updatePagination(requestPaginationModel: RequestPaginationModel) {
this._pagination.maxItems = requestPaginationModel.maxItems;
this._pagination.merge = requestPaginationModel.merge;
this._pagination.skipCount = requestPaginationModel.skipCount;
this.reload();
this.reload(requestPaginationModel.merge);
}
private syncPagination() {