Allow users to select multiple filter values for the same filter name (in the case where the value is an array)

Tom Clark · Tom Clark · commit f976447b454a · 2012-04-11T13:30:32.000-04:00
diff --git a/javascripts/sidebar.js b/javascripts/sidebar.js
@@ -35,7 +35,7 @@ $(function() {
     if(existing_filter) {
       filters = _(filters).without(existing_filter);
     } else {
-      filters = _(filters).reject(function(filter) { return filter.name === target.name; });
+      filters = _(filters).reject(function(filter) { return filter.name === target.name && filter.value === target.value; });
       filters.push(target);
     }
     redraw();

Original file line number	Diff line number	Diff line change
`@@ -35,7 +35,7 @@ $(function() {`
`35`	`35`	`if(existing_filter) {`
`36`	`36`	`filters = _(filters).without(existing_filter);`
`37`	`37`	`} else {`
`38`		`- filters = _(filters).reject(function(filter) { return filter.name === target.name; });`
	`38`	`+ filters = _(filters).reject(function(filter) { return filter.name === target.name && filter.value === target.value; });`
`39`	`39`	`filters.push(target);`
`40`	`40`	`}`
`41`	`41`	`redraw();`