Layout tweaks

mistercrunch · May 10, 2016 · 168bc4a · 168bc4a
1 parent 6c8c864
commit 168bc4a
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 8 deletions.
diff --git a/caravel/assets/visualizations/table.js b/caravel/assets/visualizations/table.js
@@ -42,7 +42,6 @@ function tableVis(slice) {
         timestampFormatter = px.timeFormatFactory(json.form_data.table_timestamp_format);
       }
 
-      var table = d3.select(slice.selector).html('').append('table')
       var div = d3.select(slice.selector);
       div.html('');
       var table = div.append('table')

diff --git a/caravel/viz.py b/caravel/viz.py
@@ -322,23 +322,25 @@ class TableViz(BaseViz):
     verbose_name = "Table View"
     credits = 'a <a href="https://github.com/airbnb/caravel">Caravel</a> original'
     fieldsets = ({
-        'label': "Chart Options",
-        'fields': (
-            'table_timestamp_format',
-            'row_limit',
-            ('include_search', None),
-        )
-    }, {
         'label': "GROUP BY",
+        'description': 'Use this section if you want a query that aggregates',
         'fields': (
             'groupby',
             'metrics',
         )
     }, {
         'label': "NOT GROUPED BY",
+        'description': 'Use this section if you want to query atomic rows',
         'fields': (
             'all_columns',
         )
+    }, {
+        'label': "Options",
+        'fields': (
+            'table_timestamp_format',
+            'row_limit',
+            ('include_search', None),
+        )
     })
     is_timeseries = False