loleaflet/src/layer/tile/TileLayer.js         |    2 +-
 loleaflet/src/map/handler/Map.TouchGesture.js |    4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)

New commits:
commit 272715a96d5f9b040a9f2e87414dfcef2c4972d6
Author:     mert <mert.tu...@collabora.com>
AuthorDate: Fri Mar 27 19:24:17 2020 +0300
Commit:     Andras Timar <andras.ti...@collabora.com>
CommitDate: Fri Mar 27 18:01:27 2020 +0100

    Fix url popups can't be clicked on browsers
    
    Change-Id: I3c2008e06370b4b48a93b68e58ad1e45a4f75d27
    Reviewed-on: https://gerrit.libreoffice.org/c/online/+/91230
    Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoff...@gmail.com>
    Reviewed-by: Andras Timar <andras.ti...@collabora.com>

diff --git a/loleaflet/src/layer/tile/TileLayer.js 
b/loleaflet/src/layer/tile/TileLayer.js
index dfe755b64..6ce377988 100644
--- a/loleaflet/src/layer/tile/TileLayer.js
+++ b/loleaflet/src/layer/tile/TileLayer.js
@@ -1119,7 +1119,7 @@ L.TileLayer = L.GridLayer.extend({
                // # for internal links
                if (!url.startsWith('#')) {
                        this._map.hyperlinkPopup = new L.Popup({className: 
'hyperlink-popup', closeButton: false, closeOnClick: false})
-                       .setContent('<div id="hyperlinkpopup" style="color: 
#000099; text-decoration: underline;">'+url+'</div>')
+                       .setContent('<a href="' + url + '" target="_blank">' + 
url + '</a>')
                        .setLatLng(position)
                        .openOn(this._map);
                }
diff --git a/loleaflet/src/map/handler/Map.TouchGesture.js 
b/loleaflet/src/map/handler/Map.TouchGesture.js
index 004d423ba..cf0041d53 100644
--- a/loleaflet/src/map/handler/Map.TouchGesture.js
+++ b/loleaflet/src/map/handler/Map.TouchGesture.js
@@ -301,9 +301,9 @@ L.Map.TouchGesture = L.Handler.extend({
                    mousePos = this._map._docLayer._latLngToTwips(latlng);
 
                // clicked a hyperlink popup - not really designed for this.
-               if (this._map.hyperlinkPopup && e.target && e.target.id === 
'hyperlinkpopup' &&
+               if (this._map.hyperlinkPopup && e.target &&
                        this._map.hyperlinkPopup._contentNode == 
e.target.parentNode)
-                       this._map.fire('hyperlinkclicked', {url: 
e.target.innerHTML});
+                       this._map.fire('hyperlinkclicked', {url: 
e.target.href});
 
                this._map.fire('closepopups');
                this._map.fire('closemobilewizard');
_______________________________________________
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

Reply via email to