mirrors/zotero
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages 1 Wiki Activity Actions

Item Tree: Reset scrollbar to top on collection change

Browse source
This commit is contained in:
Adomas Venčkauskas 2022-03-22 13:50:15 +02:00
parent 25230b0fca
commit 227b6354de
2 changed files with 3 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
chrome/content/zotero/components/windowed-list.js
View file

@ -164,7 +164,7 @@ module.exports = class {
* @param scrollOffset {Integer} offset for the top of the tree * @param scrollOffset {Integer} offset for the top of the tree
*/ */
scrollTo(scrollOffset) { scrollTo(scrollOffset) {
const maxOffset = this.itemHeight * this._getItemCount() - this.getWindowHeight(); const maxOffset = Math.max(0, this.itemHeight * this._getItemCount() - this.getWindowHeight());
scrollOffset = Math.min(Math.max(0, scrollOffset), maxOffset); scrollOffset = Math.min(Math.max(0, scrollOffset), maxOffset);
this.scrollOffset = scrollOffset; this.scrollOffset = scrollOffset;
this.targetElement.scrollTop = scrollOffset; this.targetElement.scrollTop = scrollOffset;

2
chrome/content/zotero/itemTree.jsx
View file

@ -1021,6 +1021,8 @@ var ItemTree = class ItemTree extends LibraryTree {
} }
this.forceUpdate(() => { this.forceUpdate(() => {
this.selection.selectEventsSuppressed = false; this.selection.selectEventsSuppressed = false;
// Reset scrollbar to top
this._treebox && this._treebox.scrollTo(0);
this._updateIntroText(); this._updateIntroText();
this._itemTreeLoadingDeferred.resolve(); this._itemTreeLoadingDeferred.resolve();
}); });