[#340] More cleanup of view.slickgrid.js

This commit is contained in:
John Glover
2013-04-16 13:08:06 +02:00
parent 5839ab8fa6
commit 8da456d713

View File

@@ -105,12 +105,12 @@ my.SlickGrid = Backbone.View.extend({
formatter: formatter formatter: formatter
}; };
var widthInfo = _.find(self.state.get('columnsWidth'),function(c){return c.column == field.id;}); var widthInfo = _.find(self.state.get('columnsWidth'),function(c){return c.column === field.id;});
if (widthInfo){ if (widthInfo){
column.width = widthInfo.width; column.width = widthInfo.width;
} }
var editInfo = _.find(self.state.get('columnsEditor'),function(c){return c.column == field.id;}); var editInfo = _.find(self.state.get('columnsEditor'),function(c){return c.column === field.id;});
if (editInfo){ if (editInfo){
column.editor = editInfo.editor; column.editor = editInfo.editor;
} }
@@ -119,7 +119,7 @@ my.SlickGrid = Backbone.View.extend({
// Restrict the visible columns // Restrict the visible columns
var visibleColumns = columns.filter(function(column) { var visibleColumns = columns.filter(function(column) {
return _.indexOf(self.state.get('hiddenColumns'), column.id) == -1; return _.indexOf(self.state.get('hiddenColumns'), column.id) === -1;
}); });
// Order them if there is ordering info on the state // Order them if there is ordering info on the state
@@ -136,7 +136,7 @@ my.SlickGrid = Backbone.View.extend({
// column picker // column picker
var tempHiddenColumns = []; var tempHiddenColumns = [];
for (var i = columns.length -1; i >= 0; i--){ for (var i = columns.length -1; i >= 0; i--){
if (_.indexOf(_.pluck(visibleColumns,'id'),columns[i].id) == -1){ if (_.indexOf(_.pluck(visibleColumns,'id'),columns[i].id) === -1){
tempHiddenColumns.push(columns.splice(i,1)[0]); tempHiddenColumns.push(columns.splice(i,1)[0]);
} }
} }
@@ -324,7 +324,7 @@ my.SlickGrid = Backbone.View.extend({
function updateColumn(e) { function updateColumn(e) {
var checkbox; var checkbox;
if ($(e.target).data('option') == 'autoresize') { if ($(e.target).data('option') === 'autoresize') {
var checked; var checked;
if ($(e.target).is('li')){ if ($(e.target).is('li')){
checkbox = $(e.target).find('input').first(); checkbox = $(e.target).find('input').first();