From bd06f995eefa51d76156727e16125f637c876b1e Mon Sep 17 00:00:00 2001
From: Maxime Beauchemin <maximebeauchemin@gmail.com>
Date: Sun, 28 Feb 2016 16:46:15 -0800
Subject: [PATCH 1/5] Allowing to make certain widgets immune to filter

---
 panoramix/assets/javascripts/dashboard.js    | 49 +++++++++-----------
 panoramix/models.py                          |  9 ++++
 panoramix/templates/panoramix/dashboard.html |  2 +-
 panoramix/views.py                           |  8 ++--
 4 files changed, 38 insertions(+), 30 deletions(-)

diff --git a/panoramix/assets/javascripts/dashboard.js b/panoramix/assets/javascripts/dashboard.js
index 6d57c03e99a4..78984a9d678f 100644
--- a/panoramix/assets/javascripts/dashboard.js
+++ b/panoramix/assets/javascripts/dashboard.js
@@ -10,52 +10,49 @@ require('brace/theme/crimson_editor');
 
 require('select2');
 require('../node_modules/gridster/dist/jquery.gridster.min.js');
-require('../node_modules/gridster/dist/jquery.gridster.min.js');
 
 var dashboard;
 
-var Dashboard = function(id){
-  var dash = {
-    slices: [],
-    filters: {},
-    id: id,
-    addFilter: function(slice_id, filters) {
+var Dashboard = function(obj){
+  obj['slices'] = [];
+  obj['filters'] = {};
+  obj['addFilter'] = function(slice_id, filters) {
       this.filters[slice_id] = filters;
       this.refreshExcept(slice_id);
-    },
-    readFilters: function() {
+    };
+  obj['readFilters'] = function() {
       // Returns a list of human readable active filters
       return JSON.stringify(this.filters, null, 4);
-    },
-    refreshExcept: function(slice_id) {
-      this.slices.forEach(function(slice){
-        if(slice.data.slice_id != slice_id){
-          slice.render();
-        }
-      });
-    },
-    clearFilter: function(slice_id) {
+  };
+  obj['refreshExcept'] = function(slice_id) {
+    this.slices.forEach(function(slice){
+      if(slice.data.slice_id != slice_id && obj.metadata.filter_immune_slices.indexOf(slice.data.slice_id) == -1){
+        slice.render();
+      }
+    });
+  };
+  obj['clearFilter'] = function(slice_id) {
       delete this.filters[slice_id];
       this.refreshExcept(slice_id);
-    },
-    getSlice: function(slice_id) {
+  };
+  obj['getSlice'] = function(slice_id) {
       for(var i=0; i<this.slices.length; i++){
         if (this.slices[i].data.slice_id == slice_id)
           return this.slices[i];
       }
-    }
   }
+
   $('.dashboard li.widget').each(function() {
     var data = $(this).data('slice');
-    var slice = px.Slice(data, dash);
+    var slice = px.Slice(data, obj);
     $(this).find('a.refresh').click(function(){
       slice.render();
     });
-    dash.slices.push(slice);
+    obj.slices.push(slice);
     slice.render();
   });
-  dashboard = dash;
-  return dash;
+  dashboard = obj;
+  return obj;
 }
 
 function initDashboardView() {
@@ -151,5 +148,5 @@ function initDashboardView() {
 
 $(document).ready(function() {
   initDashboardView();
-  var dashboard = Dashboard($('#dashboard_id').val());
+  var dashboard = Dashboard($('.dashboard').data('dashboard'));
 });
diff --git a/panoramix/models.py b/panoramix/models.py
index 93371a25369f..d71c7c6d6a3a 100644
--- a/panoramix/models.py
+++ b/panoramix/models.py
@@ -217,6 +217,15 @@ def css_files(self):
             l += o.css_files
         return list(set(l))
 
+    @property
+    def json_data(self):
+        return json.dumps({
+            'id': self.id,
+            'metadata': self.metadata_dejson,
+            'dashboard_title': self.dashboard_title,
+            'slug': self.slug,
+        })
+
 
 class Queryable(object):
     @property
diff --git a/panoramix/templates/panoramix/dashboard.html b/panoramix/templates/panoramix/dashboard.html
index 2db0b0371d0e..45a34626275b 100644
--- a/panoramix/templates/panoramix/dashboard.html
+++ b/panoramix/templates/panoramix/dashboard.html
@@ -9,7 +9,7 @@
 {% endblock %}
 
 {% block body %}
-<div class="dashboard">
+<div class="dashboard container-fluid" data-dashboard="{{ dashboard.json_data }}">
 
 <!-- Modal -->
 <div class="modal fade" id="css_modal" tabindex="-1" role="dialog">
diff --git a/panoramix/views.py b/panoramix/views.py
index dc22cd97f3b9..1fea4571b5a0 100644
--- a/panoramix/views.py
+++ b/panoramix/views.py
@@ -516,9 +516,11 @@ def save_dash(self, dashboard_id):
         dash = session.query(Dash).filter_by(id=dashboard_id).first()
         dash.slices = [o for o in dash.slices if o.id in slice_ids]
         dash.position_json = json.dumps(data['positions'], indent=4)
-        dash.json_metadata = json.dumps({
-            'expanded_slices': data['expanded_slices'],
-        }, indent=4)
+        md = dash.metadata_dejson
+        if 'filter_immune_slices' not in md:
+            md['filter_immune_slices'] = []
+        md['expanded_slices'] = data['expanded_slices']
+        dash.json_metadata = json.dumps(md, indent=4)
         dash.css = data['css']
         session.merge(dash)
         session.commit()

From 22327e204f134633b3a1983e1c1cc84e27b85b3d Mon Sep 17 00:00:00 2001
From: Maxime Beauchemin <maximebeauchemin@gmail.com>
Date: Sun, 28 Feb 2016 22:57:43 -0800
Subject: [PATCH 2/5] Transfering all the json in one point

---
 panoramix/assets/javascripts/dashboard.js    | 9 ++++-----
 panoramix/models.py                          | 6 ++++--
 panoramix/templates/panoramix/dashboard.html | 1 -
 3 files changed, 8 insertions(+), 8 deletions(-)

diff --git a/panoramix/assets/javascripts/dashboard.js b/panoramix/assets/javascripts/dashboard.js
index 78984a9d678f..6d65e2f60b49 100644
--- a/panoramix/assets/javascripts/dashboard.js
+++ b/panoramix/assets/javascripts/dashboard.js
@@ -14,7 +14,7 @@ require('../node_modules/gridster/dist/jquery.gridster.min.js');
 var dashboard;
 
 var Dashboard = function(obj){
-  obj['slices'] = [];
+  //obj['slices'] = [];
   obj['filters'] = {};
   obj['addFilter'] = function(slice_id, filters) {
       this.filters[slice_id] = filters;
@@ -41,16 +41,15 @@ var Dashboard = function(obj){
           return this.slices[i];
       }
   }
-
-  $('.dashboard li.widget').each(function() {
-    var data = $(this).data('slice');
+  obj.slices.forEach(function(data, i){
     var slice = px.Slice(data, obj);
-    $(this).find('a.refresh').click(function(){
+    $("#slice_" + data.slice_id).find('a.refresh').click(function(){
       slice.render();
     });
     obj.slices.push(slice);
     slice.render();
   });
+
   dashboard = obj;
   return obj;
 }
diff --git a/panoramix/models.py b/panoramix/models.py
index d71c7c6d6a3a..a535e0c50acc 100644
--- a/panoramix/models.py
+++ b/panoramix/models.py
@@ -219,12 +219,14 @@ def css_files(self):
 
     @property
     def json_data(self):
-        return json.dumps({
+        d = {
             'id': self.id,
             'metadata': self.metadata_dejson,
             'dashboard_title': self.dashboard_title,
             'slug': self.slug,
-        })
+            'slices': [slc.data for slc in self.slices],
+        }
+        return json.dumps(d)
 
 
 class Queryable(object):
diff --git a/panoramix/templates/panoramix/dashboard.html b/panoramix/templates/panoramix/dashboard.html
index 45a34626275b..71e2b15667ee 100644
--- a/panoramix/templates/panoramix/dashboard.html
+++ b/panoramix/templates/panoramix/dashboard.html
@@ -75,7 +75,6 @@ <h2>
               id="slice_{{ slice.id }}"
               slice_id="{{ slice.id }}"
               class="widget {{ slice.viz.viz_type }}"
-              data-slice="{{ slice.json_data }}"
               data-row="{{ pos.row or 1 }}"
               data-col="{{ pos.col or loop.index }}"
               data-sizex="{{ pos.size_x or 4 }}"

From 3a38c601e06e6d45ea085775d31513f93befde83 Mon Sep 17 00:00:00 2001
From: Maxime Beauchemin <maximebeauchemin@gmail.com>
Date: Sun, 28 Feb 2016 23:26:27 -0800
Subject: [PATCH 3/5] Fixing tooltips

---
 panoramix/assets/javascripts/explore.js | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/panoramix/assets/javascripts/explore.js b/panoramix/assets/javascripts/explore.js
index e5242679fa26..2618492fd8a5 100644
--- a/panoramix/assets/javascripts/explore.js
+++ b/panoramix/assets/javascripts/explore.js
@@ -8,8 +8,10 @@ var jQuery = window.jQuery = $;
 var px = require('./modules/panoramix.js');
 var d3 = require('d3');
 
-require('bootstrap');
 require('jquery-ui');
+$.widget.bridge('uitooltip', $.ui.tooltip); // Shutting down jq-ui tooltips
+require('bootstrap');
+
 require('select2');
 require('../node_modules/bootstrap-toggle/js/bootstrap-toggle.min.js');
 require('../vendor/select2.sortable.js');
@@ -109,7 +111,7 @@ function initExploreView() {
   $(".select2Sortable").select2({dropdownAutoWidth : true});
   $(".select2Sortable").select2Sortable({bindOrder: 'sortableStop'});
   $("form").show();
-  $('[data-toggle="tooltip"]').tooltip();
+  $('[data-toggle="tooltip"]').tooltip({container: 'body'});
   $(".ui-helper-hidden-accessible").remove(); // jQuery-ui 1.11+ creates a div for every tooltip
 
   function set_filters(){

From 69cfcabb72f6b955974e9c8494b4d4d4bfc11c78 Mon Sep 17 00:00:00 2001
From: Maxime Beauchemin <maximebeauchemin@gmail.com>
Date: Mon, 29 Feb 2016 16:43:13 -0800
Subject: [PATCH 4/5] Making Dashboard look better

---
 panoramix/assets/javascripts/dashboard.js | 255 +++++++++++-----------
 1 file changed, 130 insertions(+), 125 deletions(-)

diff --git a/panoramix/assets/javascripts/dashboard.js b/panoramix/assets/javascripts/dashboard.js
index 6d65e2f60b49..6808c7bada58 100644
--- a/panoramix/assets/javascripts/dashboard.js
+++ b/panoramix/assets/javascripts/dashboard.js
@@ -11,141 +11,146 @@ require('brace/theme/crimson_editor');
 require('select2');
 require('../node_modules/gridster/dist/jquery.gridster.min.js');
 
-var dashboard;
-
-var Dashboard = function(obj){
-  //obj['slices'] = [];
-  obj['filters'] = {};
-  obj['addFilter'] = function(slice_id, filters) {
-      this.filters[slice_id] = filters;
-      this.refreshExcept(slice_id);
-    };
-  obj['readFilters'] = function() {
-      // Returns a list of human readable active filters
-      return JSON.stringify(this.filters, null, 4);
-  };
-  obj['refreshExcept'] = function(slice_id) {
-    this.slices.forEach(function(slice){
-      if(slice.data.slice_id != slice_id && obj.metadata.filter_immune_slices.indexOf(slice.data.slice_id) == -1){
+var Dashboard = function(dashboardData){
+  var dashboard = $.extend(dashboardData, {
+    filters: {},
+    init: function() {
+      this.initDashboardView();
+      var sliceObjects = [];
+      var dash = this;
+      dashboard.slices.forEach(function(data, i){
+        var slice = px.Slice(data, dash);
+        $("#slice_" + data.slice_id).find('a.refresh').click(function(){
+          slice.render();
+        });
+        sliceObjects.push(slice);
         slice.render();
-      }
-    });
-  };
-  obj['clearFilter'] = function(slice_id) {
-      delete this.filters[slice_id];
-      this.refreshExcept(slice_id);
-  };
-  obj['getSlice'] = function(slice_id) {
-      for(var i=0; i<this.slices.length; i++){
-        if (this.slices[i].data.slice_id == slice_id)
-          return this.slices[i];
-      }
-  }
-  obj.slices.forEach(function(data, i){
-    var slice = px.Slice(data, obj);
-    $("#slice_" + data.slice_id).find('a.refresh').click(function(){
-      slice.render();
-    });
-    obj.slices.push(slice);
-    slice.render();
-  });
-
-  dashboard = obj;
-  return obj;
-}
-
-function initDashboardView() {
-  var gridster = $(".gridster ul").gridster({
-    widget_margins: [5, 5],
-    widget_base_dimensions: [100, 100],
-    draggable: {
-      handle: '.drag',
+      });
+      this.slices = sliceObjects;
     },
-    resize: {
-      enabled: true,
-      stop: function(e, ui, element) {
-        var slice_data = $(element).data('slice');
-        dashboard.getSlice(slice_data.slice_id).resize();
-      }
+    addFilter: function(slice_id, filters) {
+        console.log([slice_id, filters]);
+        this.filters[slice_id] = filters;
+        this.refreshExcept(slice_id);
     },
-    serialize_params: function(_w, wgd) {
-      return {
-        slice_id: $(_w).attr('slice_id'),
-        col: wgd.col,
-        row: wgd.row,
-        size_x: wgd.size_x,
-        size_y: wgd.size_y
-      };
+    readFilters: function() {
+        // Returns a list of human readable active filters
+        return JSON.stringify(this.filters, null, 4);
     },
-  }).data('gridster');
-  $("div.gridster").css('visibility', 'visible');
-  $("#savedash").click(function() {
-    var expanded_slices = {};
-    $.each($(".slice_info"), function(i, d){
-      var widget = $(this).parents('.widget');
-      var slice_description = widget.find('.slice_description');
-      if(slice_description.is(":visible"))
-        expanded_slices[$(d).attr('slice_id')] = true;
-    });
-    var data = {
-        positions: gridster.serialize(),
-        css: editor.getValue(),
-        expanded_slices: expanded_slices,
-    };
-    $.ajax({
-      type: "POST",
-      url: '/panoramix/save_dash/' + dashboard.id + '/',
-      data: {'data': JSON.stringify(data)},
-      success: function() {alert("Saved!")},
-      error: function() {alert("Error :(")},
-    });
-  });
+    refreshExcept: function(slice_id) {
+      var immune = this.metadata.filter_immune_slices;
+      this.slices.forEach(function(slice){
+        if(slice.data.slice_id != slice_id && immune.indexOf(slice.data.slice_id) == -1){
+          slice.render();
+        }
+      });
+    },
+    clearFilter: function(slice_id) {
+        delete this.filters[slice_id];
+        this.refreshExcept(slice_id);
+    },
+    getSlice: function(slice_id) {
+        for(var i=0; i<this.slices.length; i++){
+          if (this.slices[i].data.slice_id == slice_id)
+            return this.slices[i];
+        }
+    },
+    initDashboardView: function() {
+      dashboard = this;
+      var gridster = $(".gridster ul").gridster({
+        widget_margins: [5, 5],
+        widget_base_dimensions: [100, 100],
+        draggable: {
+          handle: '.drag',
+        },
+        resize: {
+          enabled: true,
+          stop: function(e, ui, element) {
+            var slice_data = $(element).data('slice');
+            dashboard.getSlice(slice_data.slice_id).resize();
+          }
+        },
+        serialize_params: function(_w, wgd) {
+          return {
+            slice_id: $(_w).attr('slice_id'),
+            col: wgd.col,
+            row: wgd.row,
+            size_x: wgd.size_x,
+            size_y: wgd.size_y
+          };
+        },
+      }).data('gridster');
+      $("div.gridster").css('visibility', 'visible');
+      $("#savedash").click(function() {
+        var expanded_slices = {};
+        $.each($(".slice_info"), function(i, d){
+          var widget = $(this).parents('.widget');
+          var slice_description = widget.find('.slice_description');
+          if(slice_description.is(":visible"))
+            expanded_slices[$(d).attr('slice_id')] = true;
+        });
+        var data = {
+            positions: gridster.serialize(),
+            css: editor.getValue(),
+            expanded_slices: expanded_slices,
+        };
+        $.ajax({
+          type: "POST",
+          url: '/panoramix/save_dash/' + dashboard.id + '/',
+          data: {'data': JSON.stringify(data)},
+          success: function() {alert("Saved!")},
+          error: function() {alert("Error :(")},
+        });
+      });
 
-  var editor = ace.edit("dash_css");
-  editor.$blockScrolling = Infinity
+      var editor = ace.edit("dash_css");
+      editor.$blockScrolling = Infinity
 
-  editor.setTheme("ace/theme/crimson_editor");
-  editor.setOptions({
-      minLines: 16,
-      maxLines: Infinity,
-  });
-  editor.getSession().setMode("ace/mode/css");
+      editor.setTheme("ace/theme/crimson_editor");
+      editor.setOptions({
+          minLines: 16,
+          maxLines: Infinity,
+      });
+      editor.getSession().setMode("ace/mode/css");
 
-  $(".select2").select2({dropdownAutoWidth : true});
-  $("#css_template").on("change", function() {
-    var css = $(this).find('option:selected').data('css');
-    editor.setValue(css);
-    $('#dash_css').val(css);
-    $("#user_style").html(css);
-  });
-  $('#filters').click( function(){
-    alert(dashboard.readFilters());
-  });
-  $("a.closeslice").click(function() {
-    var li = $(this).parents("li");
-    gridster.remove_widget(li);
-  });
-  $(".slice_info").click(function(){
-    var widget = $(this).parents('.widget');
-    var slice_description = widget.find('.slice_description');
-    slice_description.slideToggle(500, function(){
-      widget.find('.refresh').click();
-    });
-  });
-  $("table.slice_header").mouseover(function() {
-    $(this).find("td.icons nobr").show();
-  });
-  $("table.slice_header").mouseout(function() {
-    $(this).find("td.icons nobr").hide();
-  });
-  editor.on("change", function(){
-    var css = editor.getValue();
-    $('#dash_css').val(css);
-    $("#user_style").html(css);
+      $(".select2").select2({dropdownAutoWidth : true});
+      $("#css_template").on("change", function() {
+        var css = $(this).find('option:selected').data('css');
+        editor.setValue(css);
+        $('#dash_css').val(css);
+        $("#user_style").html(css);
+      });
+      $('#filters').click( function(){
+        alert(dashboard.readFilters());
+      });
+      $("a.closeslice").click(function() {
+        var li = $(this).parents("li");
+        gridster.remove_widget(li);
+      });
+      $(".slice_info").click(function(){
+        var widget = $(this).parents('.widget');
+        var slice_description = widget.find('.slice_description');
+        slice_description.slideToggle(500, function(){
+          widget.find('.refresh').click();
+        });
+      });
+      $("table.slice_header").mouseover(function() {
+        $(this).find("td.icons nobr").show();
+      });
+      $("table.slice_header").mouseout(function() {
+        $(this).find("td.icons nobr").hide();
+      });
+      editor.on("change", function(){
+        var css = editor.getValue();
+        $('#dash_css').val(css);
+        $("#user_style").html(css);
+      });
+    }
   });
+  dashboard.init();
+  return dashboard;
 }
 
 $(document).ready(function() {
-  initDashboardView();
   var dashboard = Dashboard($('.dashboard').data('dashboard'));
 });

From be4a7a407705f82a7c87a9010ff31b4f87e27606 Mon Sep 17 00:00:00 2001
From: Maxime Beauchemin <maximebeauchemin@gmail.com>
Date: Mon, 29 Feb 2016 17:55:55 -0800
Subject: [PATCH 5/5] Linting

---
 panoramix/assets/javascripts/dashboard.js | 103 +++++++++++-----------
 1 file changed, 52 insertions(+), 51 deletions(-)

diff --git a/panoramix/assets/javascripts/dashboard.js b/panoramix/assets/javascripts/dashboard.js
index 6808c7bada58..c196e6ac2bc4 100644
--- a/panoramix/assets/javascripts/dashboard.js
+++ b/panoramix/assets/javascripts/dashboard.js
@@ -11,51 +11,52 @@ require('brace/theme/crimson_editor');
 require('select2');
 require('../node_modules/gridster/dist/jquery.gridster.min.js');
 
-var Dashboard = function(dashboardData){
+
+var Dashboard = function (dashboardData) {
   var dashboard = $.extend(dashboardData, {
     filters: {},
-    init: function() {
+    init: function () {
       this.initDashboardView();
-      var sliceObjects = [];
-      var dash = this;
-      dashboard.slices.forEach(function(data, i){
-        var slice = px.Slice(data, dash);
-        $("#slice_" + data.slice_id).find('a.refresh').click(function(){
+      var sliceObjects = [],
+        dash = this;
+        dashboard.slices.forEach(function (data) {
+          var slice = px.Slice(data, dash);
+          $("#slice_" + data.slice_id).find('a.refresh').click(function () {
+            slice.render();
+          });
+          sliceObjects.push(slice);
           slice.render();
         });
-        sliceObjects.push(slice);
-        slice.render();
-      });
-      this.slices = sliceObjects;
+        this.slices = sliceObjects;
     },
-    addFilter: function(slice_id, filters) {
-        console.log([slice_id, filters]);
-        this.filters[slice_id] = filters;
-        this.refreshExcept(slice_id);
+    addFilter: function (slice_id, filters) {
+      this.filters[slice_id] = filters;
+      this.refreshExcept(slice_id);
     },
-    readFilters: function() {
-        // Returns a list of human readable active filters
-        return JSON.stringify(this.filters, null, 4);
+    readFilters: function () {
+      // Returns a list of human readable active filters
+      return JSON.stringify(this.filters, null, 4);
     },
-    refreshExcept: function(slice_id) {
+    refreshExcept: function (slice_id) {
       var immune = this.metadata.filter_immune_slices;
-      this.slices.forEach(function(slice){
-        if(slice.data.slice_id != slice_id && immune.indexOf(slice.data.slice_id) == -1){
+      this.slices.forEach(function (slice) {
+        if(slice.data.slice_id !== slice_id && immune.indexOf(slice.data.slice_id) === -1) {
           slice.render();
         }
       });
     },
-    clearFilter: function(slice_id) {
-        delete this.filters[slice_id];
-        this.refreshExcept(slice_id);
+    clearFilter: function (slice_id) {
+      delete this.filters[slice_id];
+      this.refreshExcept(slice_id);
     },
-    getSlice: function(slice_id) {
-        for(var i=0; i<this.slices.length; i++){
-          if (this.slices[i].data.slice_id == slice_id)
-            return this.slices[i];
+    getSlice: function (slice_id) {
+      this.slices.forEach(function (slice, i) {
+        if (slice.slice_id === slice_id) {
+          return slice;
         }
+      });
     },
-    initDashboardView: function() {
+    initDashboardView: function () {
       dashboard = this;
       var gridster = $(".gridster ul").gridster({
         widget_margins: [5, 5],
@@ -65,12 +66,12 @@ var Dashboard = function(dashboardData){
         },
         resize: {
           enabled: true,
-          stop: function(e, ui, element) {
+          stop: function (e, ui, element) {
             var slice_data = $(element).data('slice');
             dashboard.getSlice(slice_data.slice_id).resize();
           }
         },
-        serialize_params: function(_w, wgd) {
+        serialize_params: function (_w, wgd) {
           return {
             slice_id: $(_w).attr('slice_id'),
             col: wgd.col,
@@ -81,66 +82,66 @@ var Dashboard = function(dashboardData){
         },
       }).data('gridster');
       $("div.gridster").css('visibility', 'visible');
-      $("#savedash").click(function() {
+      $("#savedash").click(function () {
         var expanded_slices = {};
-        $.each($(".slice_info"), function(i, d){
+        $.each($(".slice_info"), function (i, d) {
           var widget = $(this).parents('.widget');
           var slice_description = widget.find('.slice_description');
           if(slice_description.is(":visible"))
             expanded_slices[$(d).attr('slice_id')] = true;
         });
         var data = {
-            positions: gridster.serialize(),
-            css: editor.getValue(),
-            expanded_slices: expanded_slices,
+          positions: gridster.serialize(),
+          css: editor.getValue(),
+          expanded_slices: expanded_slices,
         };
         $.ajax({
           type: "POST",
           url: '/panoramix/save_dash/' + dashboard.id + '/',
           data: {'data': JSON.stringify(data)},
-          success: function() {alert("Saved!")},
-          error: function() {alert("Error :(")},
+          success: function () {alert("Saved!");},
+          error: function () {alert("Error :(");},
         });
       });
 
       var editor = ace.edit("dash_css");
-      editor.$blockScrolling = Infinity
+      editor.$blockScrolling = Infinity;
 
       editor.setTheme("ace/theme/crimson_editor");
       editor.setOptions({
-          minLines: 16,
-          maxLines: Infinity,
+        minLines: 16,
+        maxLines: Infinity,
       });
       editor.getSession().setMode("ace/mode/css");
 
       $(".select2").select2({dropdownAutoWidth : true});
-      $("#css_template").on("change", function() {
+      $("#css_template").on("change", function () {
         var css = $(this).find('option:selected').data('css');
         editor.setValue(css);
         $('#dash_css').val(css);
         $("#user_style").html(css);
       });
-      $('#filters').click( function(){
+      $('#filters').click( function () {
         alert(dashboard.readFilters());
       });
-      $("a.closeslice").click(function() {
+      $("a.closeslice").click(function () {
         var li = $(this).parents("li");
         gridster.remove_widget(li);
       });
-      $(".slice_info").click(function(){
+      $(".slice_info").click(function () {
         var widget = $(this).parents('.widget');
         var slice_description = widget.find('.slice_description');
-        slice_description.slideToggle(500, function(){
+        slice_description.slideToggle(500, function () {
           widget.find('.refresh').click();
         });
       });
-      $("table.slice_header").mouseover(function() {
+      $("table.slice_header").mouseover(function () {
         $(this).find("td.icons nobr").show();
       });
-      $("table.slice_header").mouseout(function() {
+      $("table.slice_header").mouseout(function () {
         $(this).find("td.icons nobr").hide();
       });
-      editor.on("change", function(){
+      editor.on("change", function () {
         var css = editor.getValue();
         $('#dash_css').val(css);
         $("#user_style").html(css);
@@ -149,8 +150,8 @@ var Dashboard = function(dashboardData){
   });
   dashboard.init();
   return dashboard;
-}
+};
 
-$(document).ready(function() {
+$(document).ready(function () {
   var dashboard = Dashboard($('.dashboard').data('dashboard'));
 });