Merge pull request #2663 from doug-salvati/ds2621

Export buttons shouldn't show up on non-highcharts
engaging-computing · Jun 27, 2017 · 08069bc · 08069bc
2 parents 0ba5cd0 + 0b3c714
commit 08069bc
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/app/assets/javascripts/templates/visualizations/controls/save.hbs.erb b/app/assets/javascripts/templates/visualizations/controls/save.hbs.erb
@@ -12,6 +12,7 @@
     Print Visualization
   </button>
 
+  <div id="export-controls">
   <h4 class="vis-ctrl-subtitle">Export</h4>
 
     <div class="btn-group btn-group-justified">
@@ -34,4 +35,5 @@
       </button>
       </div>
     </div>
+    </div>
 </div>
diff --git a/app/assets/javascripts/visualizations/highvis/baseVis.coffee.erb b/app/assets/javascripts/visualizations/highvis/baseVis.coffee.erb
@@ -568,7 +568,7 @@ $ ->
         # TODO - implement these for non-Highchart vises
         unless @chart?
           $('#print-vis-btn').hide()
-          $('#download-vis-btn').hide()
+          $('#export-controls').hide()
 
         globals.userPermissions()