Author:   Lars Michelsen <[email protected]>
Date:     Wed Jul 24 16:42:35 2013 +0200
Committer:   Lars Michelsen <[email protected]>
Commit-Date: Wed Jul 24 16:42:35 2013 +0200

Fixed cursor confusions while adding objects

---

 ChangeLog                                    |    2 ++
 share/frontend/nagvis-js/js/frontendHover.js |   10 +++++-----
 2 files changed, 7 insertions(+), 5 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index ad04652..cde2689 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -26,6 +26,8 @@ Frontend:
   * Changed map rendering on overview from a single bulk request to one
     request per map.
   * FIX: Fixed centering of overview tables when opening the sidebar
+  * FIX: Fixed mouse pointer confusions when adding a new object and hovering
+    another menu which then shows up the hover menu
 
 1.7.10
 Frontend:
diff --git a/share/frontend/nagvis-js/js/frontendHover.js 
b/share/frontend/nagvis-js/js/frontendHover.js
index f0adcbd..40fc860 100644
--- a/share/frontend/nagvis-js/js/frontendHover.js
+++ b/share/frontend/nagvis-js/js/frontendHover.js
@@ -56,11 +56,11 @@ function hoverHide(id) {
         _hoverTimer = null;
     }
 
-    // Change cursor to auto when hiding hover menu
-    document.body.style.cursor = 'auto';
-
     var obj = getMapObjByDomObjId(id);
     if(obj) {
+        // Change cursor to auto when hiding hover menu
+        obj.parsedObject.style.cursor = 'auto';
+
         obj.hoverX = null;
         obj.hoverY = null;
         obj = null;
@@ -77,7 +77,6 @@ function hoverShow(x, y, id) {
     var obj = getMapObjByDomObjId(id);
     obj.hoverX = x;
     obj.hoverY = y;
-    obj = null;
 
     // document.body.scrollTop does not work in IE
     var scrollTop = document.body.scrollTop ? document.body.scrollTop :
@@ -94,7 +93,8 @@ function hoverShow(x, y, id) {
     }
 
     // Change cursor to "hand" when displaying hover menu
-    document.body.style.cursor = 'pointer';
+    obj.parsedObject.style.cursor = 'pointer';
+    obj = null;
 
     // hide the menu first to avoid an "up-then-over" visual effect
     hoverMenu.style.display = 'none';


------------------------------------------------------------------------------
See everything from the browser to the database with AppDynamics
Get end-to-end visibility with application monitoring from AppDynamics
Isolate bottlenecks and diagnose root cause in seconds.
Start your free trial of AppDynamics Pro today!
http://pubads.g.doubleclick.net/gampad/clk?id=48808831&iu=/4140/ostg.clktrk
_______________________________________________
Nagvis-checkins mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/nagvis-checkins

Reply via email to