From 68a4d2c5018cf7c9b963dba5d5bd5b71865e4615 Mon Sep 17 00:00:00 2001
From: TaoK <tao@klerks.biz>
Date: Sun, 18 Nov 2012 06:16:30 -0500
Subject: [PATCH 1/4] bugfix: ":visible" selector optimization on headers
 breaks later "nth-child" selector usage.

---
 ddtf.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/ddtf.js b/ddtf.js
index 6c3fe53..25df1c4 100644
--- a/ddtf.js
+++ b/ddtf.js
@@ -11,7 +11,7 @@ $.fn.ddTableFilter = function(options) {
     var table = $(this);
     var start = new Date();
     
-    $('th:visible', table).each(function(index) {
+    $('th', table).each(function(index) {
       if($(this).hasClass('skip-filter')) return;
       var selectbox = $('<select>');
       var values = [];

From 93bd70961868bac482ee0319712230d866e29235 Mon Sep 17 00:00:00 2001
From: TaoK <tao@klerks.biz>
Date: Sun, 18 Nov 2012 06:22:18 -0500
Subject: [PATCH 2/4] new feature: "filter-container-elementid" attribute on
 "th" element sets the generated select in there instead of replacing the
 content of the "th" element.

---
 ddtf.js | 10 ++++++++--
 1 file changed, 8 insertions(+), 2 deletions(-)

diff --git a/ddtf.js b/ddtf.js
index 25df1c4..c71bed5 100644
--- a/ddtf.js
+++ b/ddtf.js
@@ -42,8 +42,14 @@ $.fn.ddTableFilter = function(options) {
         $(selectbox).append('<option value="' + this.val + '">' + this.text + '</option>')
       });
 
-      $(this).wrapInner('<div style="display:none">');
-      $(this).append(selectbox);
+      var targetID = $(this).attr('filter-container-elementid');
+      if(targetID) {
+        $("#" + targetID, table).append(selectbox);
+      }
+      else {
+        $(this).wrapInner('<div style="display:none">');
+        $(this).append(selectbox);
+      }
       
       selectbox.bind('change', {column:col}, function(event) {
         var changeStart = new Date();

From 7ff43fc1e904e51648c6171532d0a37650d57981 Mon Sep 17 00:00:00 2001
From: TaoK <tao@klerks.biz>
Date: Sun, 18 Nov 2012 06:24:40 -0500
Subject: [PATCH 3/4] new feature: "showFilterInteractions" option causes
 "ddtf-optionunavailable" class to be applied to options that are disqualified
 by other selected filters. This help prevent spinning wheels with selecting
 mutually exclusive filters, BUT it adds some (noticeable) performance
 overhead on every filter selection.

---
 ddtf.js | 39 +++++++++++++++++++++++++++++++++++++++
 1 file changed, 39 insertions(+)

diff --git a/ddtf.js b/ddtf.js
index c71bed5..4456cfd 100644
--- a/ddtf.js
+++ b/ddtf.js
@@ -90,6 +90,44 @@ $.fn.ddTableFilter = function(options) {
         }
       });
 
+      if (showFilterInteractions) {
+        //handle greying out of inapplicable filters
+        $('th', table).each(function(index) {
+          if($(this).hasClass('skip-filter')) return;
+          var values = [];
+          $('tr:not(.skip-filter)', table).each(function(rowindex) {
+            var rowValue;
+            var otherfilterHidesRow = false;
+            $('td', this).each(function(columnindex) {
+              if(columnindex === index) {
+                rowValue = $(this).attr('ddtf-value');
+              }
+              else {
+                if($(this).hasClass('ddtf-filtered'))
+                  otherfilterHidesRow = true;
+              }
+            });
+            if(!otherfilterHidesRow && $.inArray(rowValue, values) === -1)
+              values.push(rowValue);
+          });
+    
+          var targetID = $(this).attr('filter-container-elementid');
+          var $targetSelect;
+    
+          if(targetID)
+            $targetSelect = $("#" + targetID + ">select", table);
+          else
+            $targetSelect = $("select", this);
+    
+          $("option", $targetSelect).each(function(index) {
+            if($.inArray($(this).val(), values) === -1 && $(this).val() != '--all--')
+              $(this).addClass("ddtf-optionunavailable");
+            else
+              $(this).removeClass("ddtf-optionunavailable");
+          });
+        });
+      }
+        
       if($.isFunction(options.afterFilter)) {
         options.afterFilter.apply(table);
       }
@@ -127,6 +165,7 @@ $.fn.ddTableFilter.defaultOptions = {
   },
   emptyText:'--Empty--',
   sortOpt:true,
+  showFilterInteractions:false,
   debug:false,
   minOptions:2
 }

From 8bab4fbcdb262d6dbf8d4241e0212ae71a36641d Mon Sep 17 00:00:00 2001
From: TaoK <tao@klerks.biz>
Date: Sun, 18 Nov 2012 06:44:28 -0500
Subject: [PATCH 4/4] Updated instructions for new options

---
 README.md | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/README.md b/README.md
index 0d294f4..f635294 100644
--- a/README.md
+++ b/README.md
@@ -48,6 +48,14 @@ For customization, the following parameters are available:
 *  sortOpt: A boolean indicating whether or not to sort the options.  This must be true for sortOptCallback to work.  Defaults to true.
 *  debug: A boolean indicating whether or not to run in "debug" mode.  Debug mode displays performance data.
 *  minOptions: Number of options required to show a select filter.  Defaults to 2, meaning if a column only has one unique value, no select box will be shown (what would be the point?).
+*  showFilterInteractions: A boolean indicating whether or not to add a "ddtf-optionunavailable" class to filter values that will interact with other already-applied filters to result in the table being completely filtered.  This helps avoid users wasting time on useless options, but does add some overhead and therefore may not be appropriate for very large tables, especially in restricted environments (mobile). 
+
+Markup Options
+--------------------
+The plugin will also take into account the following instructions in the table markup:
+
+*  "skip-filter" class on column headers ("th" elements) will cause the plugin to avoid that column.
+*  "filter-container-elementid" attribute on column headers ("th" elements) will cause the filtering drop-down to be created in the specified container element, instead of replacing the content of the header element.
 
 Compatibility
 -------------------