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

fx-compat: Fix positioning and size of progress popups

Browse source 
In Firefox 115, overwriting `outerHeight` caused it to not report the
correct value later, and whatever issue were were working around seems
to have been resolved since Firefox 3.

https://forums.zotero.org/discussion/113633/zotero-7-beta-bug-report-the-progresswindow-shows-at-incorrect-position
This commit is contained in:
Dan Stillman 2024-04-23 04:26:42 -04:00
parent 784ccdd996
commit 60fd0829b0
2 changed files with 4 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
chrome/content/zotero/progressWindow.xhtml
View file

@ -35,7 +35,7 @@
title="&zotero.progress.title;" title="&zotero.progress.title;"
no-titlebar-icon="true" no-titlebar-icon="true"
drawintitlebar-platforms="win,linux" drawintitlebar-platforms="win,linux"
width="300" style="min-width: 300px"
windowtype="alert:alert"> windowtype="alert:alert">
<vbox id="zotero-progress-text-box" flex="1" style="padding: 10px"> <vbox id="zotero-progress-text-box" flex="1" style="padding: 10px">

6
chrome/content/zotero/xpcom/progressWindow.js
View file

@ -45,6 +45,9 @@ Zotero.ProgressWindowSet = new function() {
} }
/**
* @progressWin {Window} - DOM Window
*/
function tile(progressWin) { function tile(progressWin) {
var parent = progressWin.opener; var parent = progressWin.opener;
var y_sub = null; var y_sub = null;
@ -477,9 +480,6 @@ Zotero.ProgressWindow = function(options = {}) {
} }
function _move() { function _move() {
// sizeToContent() fails in FF3 with multiple lines
// if we don't change the height
_progressWindow.outerHeight = _progressWindow.outerHeight + 1;
_progressWindow.sizeToContent(); _progressWindow.sizeToContent();
Zotero.ProgressWindowSet.tile(_progressWindow); Zotero.ProgressWindowSet.tile(_progressWindow);
} }