From 39a6c1eb7461fdde8c201a8d454432aee7569705 Mon Sep 17 00:00:00 2001
From: Rufus Pollock <rufus.pollock@okfn.org>
Date: Sun, 15 Apr 2012 16:35:33 +0100
Subject: [PATCH] [refactor][xs]: rename DataGrid to Grid for simplicity and
 consistency with file name.

---
 app/js/app.js          |  2 +-
 index.html             |  4 ++--
 src/view-grid.js       | 14 +++++++-------
 src/view.js            |  6 +++---
 test/view-grid.test.js | 10 +++++-----
 5 files changed, 18 insertions(+), 18 deletions(-)

diff --git a/app/js/app.js b/app/js/app.js
index bdf612f9..807b3c63 100755
--- a/app/js/app.js
+++ b/app/js/app.js
@@ -51,7 +51,7 @@ function standardViews(dataset) {
     {
       id: 'grid',
       label: 'Grid',
-      view: new recline.View.DataGrid({
+      view: new recline.View.Grid({
         model: dataset
       })
     },
diff --git a/index.html b/index.html
index 7eb729cb..30c80bf8 100644
--- a/index.html
+++ b/index.html
@@ -251,7 +251,7 @@ also easily write your own). Each view holds a pointer to a Dataset:</p>
 <ul>
   <li>DataExplorer: the parent view which manages the overall app and sets up
   sub views.</li>
-  <li>DataGrid: the data grid view.</li>
+  <li>Grid: the data grid view.</li>
   <li>Graph: a simple graphing view using <a
     href="http://code.google.com/p/flot/">Flot</a>.</li>
   <li>Map: a map view using <a href="http://leaflet.cloudmade.com/">Leaflet</a>.</li>
@@ -270,7 +270,7 @@ are useful:</p>
 <ul>
   <li><a href="docs/model.html">Models</a></li>
   <li><a href="docs/view.html">DataExplorer View (plus common view code)</a></li>
-  <li><a href="docs/view-grid.html">DataGrid View</a></li>
+  <li><a href="docs/view-grid.html">(Data) Grid View</a></li>
   <li><a href="docs/view-graph.html">Graph View (based on Flot)</a></li>
   <li><a href="docs/view-map.html">Map View (based on Leaflet)</a></li>
 </ul>
diff --git a/src/view-grid.js b/src/view-grid.js
index 02082455..cb959f18 100644
--- a/src/view-grid.js
+++ b/src/view-grid.js
@@ -4,12 +4,12 @@ this.recline = this.recline || {};
 this.recline.View = this.recline.View || {};
 
 (function($, my) {
-// ## DataGrid
+// ## (Data) Grid Dataset View
 //
 // Provides a tabular view on a Dataset.
 //
 // Initialize it with a `recline.Model.Dataset`.
-my.DataGrid = Backbone.View.extend({
+my.Grid = Backbone.View.extend({
   tagName:  "div",
   className: "recline-grid-container",
 
@@ -211,7 +211,7 @@ my.DataGrid = Backbone.View.extend({
     this.model.currentDocuments.forEach(function(doc) {
       var tr = $('<tr />');
       self.el.find('tbody').append(tr);
-      var newView = new my.DataGridRow({
+      var newView = new my.GridRow({
           model: doc,
           el: tr,
           fields: self.fields
@@ -223,22 +223,22 @@ my.DataGrid = Backbone.View.extend({
   }
 });
 
-// ## DataGridRow View for rendering an individual document.
+// ## GridRow View for rendering an individual document.
 //
 // Since we want this to update in place it is up to creator to provider the element to attach to.
 //
-// In addition you *must* pass in a FieldList in the constructor options. This should be list of fields for the DataGrid.
+// In addition you *must* pass in a FieldList in the constructor options. This should be list of fields for the Grid.
 //
 // Example:
 //
 // <pre>
-// var row = new DataGridRow({
+// var row = new GridRow({
 //   model: dataset-document,
 //     el: dom-element,
 //     fields: mydatasets.fields // a FieldList object
 //   });
 // </pre>
-my.DataGridRow = Backbone.View.extend({
+my.GridRow = Backbone.View.extend({
   initialize: function(initData) {
     _.bindAll(this, 'render');
     this._fields = initData.fields;
diff --git a/src/view.js b/src/view.js
index 368bc95b..b78e1901 100644
--- a/src/view.js
+++ b/src/view.js
@@ -38,14 +38,14 @@ this.recline.View = this.recline.View || {};
 //
 // **views**: (optional) the dataset views (Grid, Graph etc) for
 // DataExplorer to show. This is an array of view hashes. If not provided
-// just initialize a DataGrid with id 'grid'. Example:
+// just initialize a Grid with id 'grid'. Example:
 //
 // <pre>
 // var views = [
 //   {
 //     id: 'grid', // used for routing
 //     label: 'Grid', // used for view switcher
-//     view: new recline.View.DataGrid({
+//     view: new recline.View.Grid({
 //       model: dataset
 //     })
 //   },
@@ -107,7 +107,7 @@ my.DataExplorer = Backbone.View.extend({
       this.pageViews = [{
         id: 'grid',
         label: 'Grid',
-        view: new my.DataGrid({
+        view: new my.Grid({
             model: this.model
           })
       }];
diff --git a/test/view-grid.test.js b/test/view-grid.test.js
index 750cca2a..b415acde 100644
--- a/test/view-grid.test.js
+++ b/test/view-grid.test.js
@@ -1,10 +1,10 @@
 (function ($) {
 
-module("View - DataGrid");
+module("View - Grid");
 
 test('menu - hideColumn', function () {
   var dataset = Fixture.getDataset();
-  var view = new recline.View.DataGrid({
+  var view = new recline.View.Grid({
     model: dataset
   });
   $('.fixtures .test-datatable').append(view.el);
@@ -22,7 +22,7 @@ test('menu - hideColumn', function () {
 
 test('state', function () {
   var dataset = Fixture.getDataset();
-  var view = new recline.View.DataGrid({
+  var view = new recline.View.Grid({
     model: dataset,
     state: {
       hiddenFields: ['z']
@@ -35,7 +35,7 @@ test('state', function () {
   view.remove();
 });
 
-test('new DataGridRow View', function () {
+test('new GridRow View', function () {
   var $el = $('<tr />');
   $('.fixtures .test-datatable').append($el);
   var doc = new recline.Model.Document({
@@ -43,7 +43,7 @@ test('new DataGridRow View', function () {
     'b': '2',
     'a': '1'
     });
-  var view = new recline.View.DataGridRow({
+  var view = new recline.View.GridRow({
     model: doc
     , el: $el
     , fields: new recline.Model.FieldList([{id: 'a'}, {id: 'b'}])