loleaflet/js/w2ui-1.5.rc1.js | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-)
New commits: commit 01540059319258778dde4d5c9b26e811b979e1c2 Author: Ashod Nakashian <ashod.nakash...@collabora.co.uk> AuthorDate: Tue Feb 5 08:06:53 2019 -0500 Commit: Szymon Kłos <szymon.k...@collabora.com> CommitDate: Mon Feb 25 13:55:52 2019 +0100 leaflet: account for rounding error when scrolling toolbar On some mobile browsers (Chrome) we get the maximum scroll value that is 0.5 less than the real maximum, so we always think we still have room to scroll more right. This has the undesirable side-effect of keeping the >> button visible, even when there is no way to scroll further right. This allows for at most 1 pixel error. Change-Id: Ie2e8902f1e2f4f358f210e9b0236422bc4f1da00 Reviewed-on: https://gerrit.libreoffice.org/67995 Reviewed-by: Szymon Kłos <szymon.k...@collabora.com> Tested-by: Szymon Kłos <szymon.k...@collabora.com> diff --git a/loleaflet/js/w2ui-1.5.rc1.js b/loleaflet/js/w2ui-1.5.rc1.js index 023587e95..fbbcc3d63 100644 --- a/loleaflet/js/w2ui-1.5.rc1.js +++ b/loleaflet/js/w2ui-1.5.rc1.js @@ -13086,7 +13086,8 @@ var w2prompt = function (label, title, callBack) { if (scrollBox.scrollLeft() > 0) { box.find('.w2ui-scroll-left').show(); } - if (scrollBox.scrollLeft() < scrollBox.find(':first').outerWidth() - scrollBox.outerWidth()) { + // Allow for rounding errors (on mobile, the max scrollLeft we get is 0.5 less than the real max). + if (scrollBox.scrollLeft() < scrollBox.find(':first').outerWidth() - scrollBox.outerWidth() - 1) { box.find('.w2ui-scroll-right').show(); } } @@ -13790,7 +13791,8 @@ var w2prompt = function (label, title, callBack) { if (scrollBox.scrollLeft() > 0) { box.find('.w2ui-scroll-left').show(); } - if (scrollBox.scrollLeft() < scrollBox.find(':first').outerWidth() - scrollBox.outerWidth()) { + // Allow for rounding errors (on mobile, the max scrollLeft we get is 0.5 less than the real max). + if (scrollBox.scrollLeft() < scrollBox.find(':first').outerWidth() - scrollBox.outerWidth() - 1) { box.find('.w2ui-scroll-right').show(); } } _______________________________________________ Libreoffice-commits mailing list libreoffice-comm...@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits