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

Fix recursion error when toggling/resizing sidebar in reader window

Browse source 
Fixes #3379
This commit is contained in:
Martynas Bagdonas 2023-10-19 18:15:51 +01:00
parent c27bac2ad6
commit 7e82cc5ffd
1 changed files with 6 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

12
chrome/content/zotero/xpcom/reader.js
View file

@ -329,13 +329,13 @@ class ReaderInstance {
}
},
onToggleSidebar: (open) => {
if (this._onToggleSidebar) {
this._onToggleSidebar(open);
if (this._onToggleSidebarCallback) {
this._onToggleSidebarCallback(open);
}
},
onChangeSidebarWidth: (width) => {
if (this._onChangeSidebarWidth) {
this._onChangeSidebarWidth(width);
if (this._onChangeSidebarWidthCallback) {
this._onChangeSidebarWidthCallback(width);
}
},
onFocusSplitButton: () => {
@ -993,8 +993,8 @@ class ReaderTab extends ReaderInstance {
this._sidebarOpen = options.sidebarOpen;
this._bottomPlaceholderHeight = options.bottomPlaceholderHeight;
this._showItemPaneToggle = true;
this._onToggleSidebar = options.onToggleSidebar;
this._onChangeSidebarWidth = options.onChangeSidebarWidth;
this._onToggleSidebarCallback = options.onToggleSidebar;
this._onChangeSidebarWidthCallback = options.onChangeSidebarWidth;
this._window = Services.wm.getMostRecentWindow('navigator:browser');
let { id, container } = this._window.Zotero_Tabs.add({
id: options.tabID,