Merge pull request #208 from leonard-thieu/show-hide-same-event

Show and hide triggers can now be set to the same event

Author: 45kb

index.html

@@ -423,6 +423,24 @@
       </span>
     </div>
 
+    <div>
+      <span tooltips
+        tooltip-show-trigger="click"
+        tooltip-hide-trigger="click"
+        tooltip-template="Click again to close">
+        Tooltip with same trigger (click) for show and hide
+      </span>
+    </div>
+
+    <div>
+      <span tooltips
+        tooltip-show-trigger="mouseenter"
+        tooltip-hide-trigger="mouseenter"
+        tooltip-template="Mouseover again to close">
+        Tooltip with same trigger (mouseover) for show and hide
+      </span>
+    </div>
+
     <script type="text/javascript" src="https://cdnjs.cloudflare.com/ajax/libs/angular.js/1.6.0/angular.js"></script>
     <script type="text/javascript" src="lib/angular-tooltips.js"></script>
     <script type="text/javascript" src="demo/js/index.js"></script>

lib/angular-tooltips.js

@@ -328,6 +328,11 @@
           }
           , onTooltipShow = function onTooltipShow(event) {
 
+            if (event && !tooltipElement.hasClass('active')) {
+              
+              event.stopImmediatePropagation();
+            }
+
             tipElement.addClass('_hidden');
             if ($attrs.tooltipSmart) {
 
@@ -448,7 +453,12 @@
               }
             }
           }
-          , onTooltipHide = function onTooltipHide() {
+          , onTooltipHide = function onTooltipHide(event) {
+
+            if (event && tooltipElement.hasClass('active')) {
+  
+              event.stopImmediatePropagation();
+            }
 
             if ($attrs.tooltipAppendToBody) {