Merge pull request #16 from dolin5/attribute-table-tab-closure-and-sort

Attribute table tab closure and sort

Author: tmcgee

widgets/AttributesTable.js

@@ -135,8 +135,13 @@ define([
             if (!options.id) {
                 options.id = 'attrTab-' + options.topicID;
             }
+            if (typeof options.closable == 'undefined'){
+                options.closable = true;
+            }
+            if (typeof options.confirmClose == 'undefined'){
+                options.confirmClose = true;
+            }
             options.map = this.map;
-            options.closable = true;
             options.sidebarID = this.sidebarID;
 
             if (this.useTabs) {
@@ -154,7 +159,7 @@ define([
                     tab.startup();
                     tabs.addChild(tab);
                     tab.onClose = lang.hitch(tab, function () {
-                        var close = confirm('Do you really want to close this tab?');
+                        var close = this.confirmClose ? confirm('Do you really want to close this tab?') : true;
                         if (close && this.clearAll) {
                             this.clearAll();
                         }

widgets/AttributesTable/README.md

@@ -93,6 +93,11 @@ define({
             // other instances of attributes table
             topicID: 'query',
 
+            // allow tabs to be closed
+            // confirm tab closure
+            closable: true,
+            confirmClose: true,
+
             queryOptions: {
                 // parameters for the query
                 queryParameters: {
@@ -225,7 +230,7 @@ gridOptions: {
     columns: [],
 
     /*
-        no sort
+        no sort, use sort: 'inherit' to use the order of the query results.
     */
     sort: [],
 

widgets/AttributesTable/_GridMixin.js

@@ -251,6 +251,10 @@ define([
 
             // set the sort
             var sort = this.gridOptions.sort || [];
+            // sort === 'inherit'? use query result order
+            if (typeof sort === 'string' && sort.toLowerCase() === 'inherit'){
+                return;
+            }
             // no sort? use the first column
             if (sort.length < 1 && columns && columns.length > 0) {
                 sort = [