Since Leaflet 1.0 there is buildin <a href="https://leafletjs.com/reference-1.0.0.html

<a class="user-mention notranslate" data-hovercard-type="user" data-hover

Hi <a class="user-mention notranslate" data-hovercard-type="user" data-hovercard-url="

<a class="user-mention notranslate" data-hovercard-type="user" data-hovercard-url="/us

Consider using Leaflet.Tooltip about ingress-intel-total-conversion HOT 7 OPEN

iitc-ce commented on May 20, 2024

Consider using Leaflet.Tooltip

from ingress-intel-total-conversion.

Comments (7)

DanielOndiordna commented on May 20, 2024 1

My player tracker add-on plugin already adds permanent player names as labels to the markers.
https://softspot.nl/ingress/plugindetails/iitc-plugin-player-tracker-addon
This plugin adds Leaflet.iconlabel:
https://github.com/jacobtoye/Leaflet.iconlabel

from ingress-intel-total-conversion.

xscreach commented on May 20, 2024 1

@DanielOndiordna thanks! I still think this would be a useful optional feature to contribute into the official plugin :) also I think it would be useful to display the machina movement (e.g. with white icon), also optionally.

Imho a good idea 👍
Just note that Machina movement is handled by machina-tracker plugin 😉

from ingress-intel-total-conversion.

johnd0e commented on May 20, 2024

For example, it could be used in player-tracker in order to provide (optionally) permanent labels for players' nicks.

player-tracker code is not great. E.g. this part currently seems do nothing:

ingress-intel-total-conversion/plugins/player-activity-tracker.user.js

Lines 50 to 55 in 256f9c5

 if(obj.layer === plugin.playerTracker.drawnTracesEnl || obj.layer === plugin.playerTracker.drawnTracesRes) { 

 obj.layer.eachLayer(function(marker) { 

 if(marker._icon) window.setupTooltips($(marker._icon)); 

 }); 

 } 

 });

Similar code is here

ingress-intel-total-conversion/plugins/player-activity-tracker.user.js

Lines 404 to 407 in 256f9c5

 // jQueryUI doesn’t automatically notice the new markers 

 if (!isTouchDev) { 

 window.setupTooltips($(m._icon)); 

 }

And here and there I see workaround if form of force calling closeIconTooltips

ingress-intel-total-conversion/plugins/player-activity-tracker.user.js

Lines 81 to 89 in 256f9c5

 // force close all open tooltips before markers are cleared 

 window.plugin.playerTracker.closeIconTooltips = function() { 

 plugin.playerTracker.drawnTracesRes.eachLayer(function(layer) { 

 if ($(layer._icon)) { $(layer._icon).tooltip('close');} 

 }); 

 plugin.playerTracker.drawnTracesEnl.eachLayer(function(layer) { 

 if ($(layer._icon)) { $(layer._icon).tooltip('close');} 

 }); 

 }

I suppose that using leaflet native tooltips instead of jQueryUI would be much easier and do not require any special handling, we just need bindTooltip here:

ingress-intel-total-conversion/plugins/player-activity-tracker.user.js

Line 389 in 256f9c5

 var m = L.marker(gllfe(last), {icon: icon, referenceToPortal: closestPortal, opacity: absOpacity, desc: popup[0], title: tooltip}); 

But as for initial intention (bindTooltip with permanent: true option on every marker) then we should consider to apply special tooltips styling (default is not good for dense areas).

from ingress-intel-total-conversion.

johnd0e commented on May 20, 2024

Also we could use tooltips with portal markers, to display title if available (or some other info in concise form).

To avoid tooltip overlapping: https://github.com/ZijingPeng/leaflet-tooltip-layout

from ingress-intel-total-conversion.

Gollam commented on May 20, 2024

Hi @modos189, I just had the idea of showing agent names in player tracker, and I found this open issue. Do you think I should try to write some code for it?

from ingress-intel-total-conversion.

Gollam commented on May 20, 2024

@DanielOndiordna thanks! I still think this would be a useful optional feature to contribute into the official plugin :) also I think it would be useful to display the machina movement (e.g. with white icon), also optionally.

from ingress-intel-total-conversion.

Gollam commented on May 20, 2024

@xscreach thanks! I'll check it.

from ingress-intel-total-conversion.

Consider using Leaflet.Tooltip about ingress-intel-total-conversion HOT 7 OPEN

Comments (7)

Related Issues (20)

Recommend Projects

React

Vue.js

Typescript

TensorFlow

Django

Laravel

D3

Recommend Topics

javascript

web

server

Machine learning

Visualization

Game

Recommend Org

Facebook

Microsoft

Google

Alibaba

D3

Tencent

	if(obj.layer === plugin.playerTracker.drawnTracesEnl \|\| obj.layer === plugin.playerTracker.drawnTracesRes) {
	obj.layer.eachLayer(function(marker) {
	if(marker._icon) window.setupTooltips($(marker._icon));
	});
	}
	});

	// jQueryUI doesn’t automatically notice the new markers
	if (!isTouchDev) {
	window.setupTooltips($(m._icon));
	}

	// force close all open tooltips before markers are cleared
	window.plugin.playerTracker.closeIconTooltips = function() {
	plugin.playerTracker.drawnTracesRes.eachLayer(function(layer) {
	if ($(layer._icon)) { $(layer._icon).tooltip('close');}
	});
	plugin.playerTracker.drawnTracesEnl.eachLayer(function(layer) {
	if ($(layer._icon)) { $(layer._icon).tooltip('close');}
	});
	}