loleaflet/src/control/Control.LokDialog.js | 12 +++++++++++- 1 file changed, 11 insertions(+), 1 deletion(-)
New commits: commit 859915699a650e2652c483c7e8d616fe0979cf57 Author: Tor Lillqvist <t...@collabora.com> AuthorDate: Fri Apr 19 18:11:16 2019 +0300 Commit: Tor Lillqvist <t...@collabora.com> CommitDate: Fri Apr 19 18:15:48 2019 +0300 tdf#124235: Make the combo box list be attached to the widget Don't multiply the offset added to 'top' by the dpi scale factor in the iOS app. I see the same problem in Safari on a Retina Mac against normal online, too, so possibly the multiplication should never be done? Change-Id: Id63d1d556e8b89612deacf01a734825f1940a5fa diff --git a/loleaflet/src/control/Control.LokDialog.js b/loleaflet/src/control/Control.LokDialog.js index 1836bf7a2..3282b9d2e 100644 --- a/loleaflet/src/control/Control.LokDialog.js +++ b/loleaflet/src/control/Control.LokDialog.js @@ -658,7 +658,17 @@ L.Control.LokDialog = L.Control.extend({ var dialogTitle = $('.lokdialog_notitle'); if (dialogTitle != null && dialogTitle.length == 0) { var dialogTitleBar = $('.ui-dialog-titlebar'); - top += dialogTitleBar.outerHeight() * L.getDpiScaleFactor(); + // 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 Retuna 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(); + } } floatingCanvas.id = strId + '-floating'; _______________________________________________ Libreoffice-commits mailing list libreoffice-comm...@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits