Don't show an empty tooltip

webtrike · May 16, 2017 · 4608488 · 4608488
1 parent 0ed337e
commit 4608488
Showing 1 changed file with 12 additions and 1 deletion.
diff --git a/src/Tooltip.js b/src/Tooltip.js
@@ -22,6 +22,7 @@ L.Draw.Tooltip = L.Class.extend({
 	initialize: function (map) {
 		this._map = map;
 		this._popupPane = map._panes.popupPane;
+		this._visible = false;
 
 		this._container = map.options.drawControlTooltips ?
 			L.DomUtil.create('div', 'leaflet-draw-tooltip', this._popupPane) : null;
@@ -64,6 +65,14 @@ L.Draw.Tooltip = L.Class.extend({
 			'<span class="leaflet-draw-tooltip-subtext">' + labelText.subtext + '</span>' + '<br />' : '') +
 			'<span>' + labelText.text + '</span>';
 
+		if (!labelText.text && !labelText.subtext) {
+			this._visible = false;
+			this._container.style.visibility = 'hidden';
+		} else {
+			this._visible = true;
+			this._container.style.visibility = 'inherit';
+		}
+
 		return this;
 	},
 
@@ -74,7 +83,9 @@ L.Draw.Tooltip = L.Class.extend({
 			tooltipContainer = this._container;
 
 		if (this._container) {
-			tooltipContainer.style.visibility = 'inherit';
+			if (this._visible) {
+				tooltipContainer.style.visibility = 'inherit';
+			}
 			L.DomUtil.setPosition(tooltipContainer, pos);
 		}