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

Fix reader popups being covered by contextPane in Stacked View mode

Browse source 
Fixes #3542
This commit is contained in:
Martynas Bagdonas 2024-07-16 14:33:11 +03:00
parent 220bb43830
commit a46f1bcbbb
2 changed files with 14 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

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

@ -521,6 +521,18 @@ class ReaderInstance {
},
onTextSelectionAnnotationModeChange: (mode) => {
Zotero.Prefs.set('reader.textSelectionAnnotationMode', mode);
},
onBringReaderToFront: (bring) => {
// Temporary bring reader iframe to front to make sure popups and context menus
// aren't overlapped by contextPane, in Stacked View mode
if (bring) {
if (Zotero.Prefs.get('layout') === 'stacked') {
this._iframe.parentElement.style.zIndex = 1;
}
}
else {
this._iframe.parentElement.style.zIndex = 'unset';
}
}
}, this._iframeWindow, { cloneFunctions: true }));
@ -1068,6 +1080,7 @@ class ReaderTab extends ReaderInstance {
this._iframe.setAttribute('class', 'reader');
this._iframe.setAttribute('flex', '1');
this._iframe.setAttribute('type', 'content');
this._iframe.setAttribute('transparent', 'true');
this._iframe.setAttribute('src', 'resource://zotero/reader/reader.html');
this._tabContainer.appendChild(this._iframe);
this._iframe.docShell.windowDraggingAllowed = true;

2
reader

@ -1 +1 @@
Subproject commit de469e74149c5b0412cd55d03fdd4c0fbdb92734
Subproject commit 58a386301a62a0413d5a91471e6036740acc730f