[Libreoffice-commits] online.git: loleaflet/src

Libreoffice Gerrit user logerrit at kemper.freedesktop.org
Tue Apr 23 02:52:24 UTC 2019


 loleaflet/src/control/Control.LokDialog.js |   12 +-----------
 1 file changed, 1 insertion(+), 11 deletions(-)

New commits:
commit ede2c3432f1d25c1c74b94f8fd5686fd59750e7b
Author:     Tor Lillqvist <tml at collabora.com>
AuthorDate: Sat Apr 20 23:32:37 2019 +0300
Commit:     Ashod Nakashian <ashnakash at gmail.com>
CommitDate: Tue Apr 23 04:52:05 2019 +0200

    tdf#124235: Fix the same problem on all platforms the same way
    
    Change-Id: Ie7ede59841898a0738af7e44a3c0fe89db1cd3ee
    Reviewed-on: https://gerrit.libreoffice.org/71023
    Reviewed-by: Ashod Nakashian <ashnakash at gmail.com>
    Tested-by: Ashod Nakashian <ashnakash at gmail.com>

diff --git a/loleaflet/src/control/Control.LokDialog.js b/loleaflet/src/control/Control.LokDialog.js
index a58fd6008..ce6940698 100644
--- a/loleaflet/src/control/Control.LokDialog.js
+++ b/loleaflet/src/control/Control.LokDialog.js
@@ -658,17 +658,7 @@ L.Control.LokDialog = L.Control.extend({
 		var dialogTitle = $('.lokdialog_notitle');
 		if (dialogTitle != null && dialogTitle.length == 0) {
 			var dialogTitleBar = $('.ui-dialog-titlebar');
-			// tdf#124235: At least in the iOS app, multiplying with
-			// L.getDpiScaleFactor() below causes the child of a combo box to be
-			// displaced from the fixed part. I see the same problem also when using
-			// Safari on a Retina Mac against normal online. But as I don't know whether
-			// it happens also for other browsers on other platforms on hidpi displays,
-			// I will fix this for the iOS app only for now.
-			if (!window.ThisIsTheiOSApp) {
-				top += dialogTitleBar.outerHeight() * L.getDpiScaleFactor();
-			} else {
-				top += dialogTitleBar.outerHeight();
-			}
+			top += dialogTitleBar.outerHeight();
 		}
 
 		floatingCanvas.id = strId + '-floating';


More information about the Libreoffice-commits mailing list