[Nagvis-checkins] SF.net Git: nagvis: Bugfix: Fixed flickering hover menu on overview page when sidebar opened

[lars <la...@us...>]

Author:   Lars Michelsen <lm...@la...>
Date:     Wed Nov 28 17:35:15 2012 +0100
Committer:   Lars Michelsen <lm...@la...>
Commit-Date: Wed Nov 28 17:35:15 2012 +0100

Bugfix: Fixed flickering hover menu on overview page when sidebar opened

---

 ChangeLog                                    |    1 +
 share/frontend/nagvis-js/js/frontendHover.js |    4 ++--
 2 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index 3ea3fbb..1e2fdfb 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -7,6 +7,7 @@ Frontend:
   * Hiding HTTP status error messages automatically on next successful request
   * Bugfix: Fixed "img" not defined js error in IE8 (Please note: Zoom
     function does not work in IE8 - IE8 event handling seems to be broken)
+  * Bugfix: Fixed flickering hover menu on overview page when sidebar opened
 
 Geomap:
   * New option source_type to configure a csv file or the used backend as
diff --git a/share/frontend/nagvis-js/js/frontendHover.js b/share/frontend/nagvis-js/js/frontendHover.js
index 8fa8deb..4f8ace0 100644
--- a/share/frontend/nagvis-js/js/frontendHover.js
+++ b/share/frontend/nagvis-js/js/frontendHover.js
@@ -98,8 +98,8 @@ function hoverShow(x, y, id) {
 
     // hide the menu first to avoid an "up-then-over" visual effect
     hoverMenu.style.display = 'none';
-    hoverMenu.style.left = x + scrollLeft + hoverSpacer - getSidebarWidth() + 'px';
-    hoverMenu.style.top = y + scrollTop + hoverSpacer - getHeaderHeight() + 'px';
+    hoverMenu.style.left = (x + scrollLeft + hoverSpacer) + 'px';
+    hoverMenu.style.top = (y + scrollTop + hoverSpacer) + 'px';
     if(isIE) {
         hoverMenu.style.width = '0px';
     } else {