From a046fabe9d24efeae7f4e0403a7598701557efac Mon Sep 17 00:00:00 2001
From: Oliver Gorwits <oliver@cpan.org>
Date: Sun, 9 Dec 2012 18:32:07 +0000
Subject: [PATCH] bug fix netmap help icon show/hide

---
 Netdisco/views/ajax/device/netmap.tt |  3 ---
 Netdisco/views/js/device.js          | 10 +++++++++-
 2 files changed, 9 insertions(+), 4 deletions(-)

diff --git a/Netdisco/views/ajax/device/netmap.tt b/Netdisco/views/ajax/device/netmap.tt
index 7b7aae00..7dab3688 100644
--- a/Netdisco/views/ajax/device/netmap.tt
+++ b/Netdisco/views/ajax/device/netmap.tt
@@ -1,8 +1,5 @@
 <script>
 
-// show our help icon
-d3.select('#netmap_help_img').style('display', 'inline');
-
 var winHeight = window.innerHeight;
 var winWidth  = window.innerWidth;
 
diff --git a/Netdisco/views/js/device.js b/Netdisco/views/js/device.js
index 19366395..0cec793e 100644
--- a/Netdisco/views/js/device.js
+++ b/Netdisco/views/js/device.js
@@ -2,7 +2,7 @@
   // ajax content is loaded
   var path = 'device';
 
-  function inner_view_processing() {
+  function inner_view_processing(tab) {
     // VLANs column list collapser trigger
     // it's a bit of a faff because we can't easily use Bootstrap's collapser
     $('.nd_collapse_vlans').toggle(function() {
@@ -72,6 +72,14 @@
       }
     });
 
+    // show or hide netmap help button
+    if (tab == 'netmap') {
+      $('#netmap_help_img').show();
+    }
+    else {
+      $('#netmap_help_img').hide();
+    }
+
     // activate tooltips and popovers
     $("[rel=tooltip]").tooltip({live: true});
     $("[rel=popover]").popover({live: true});