summaryrefslogtreecommitdiffstats
path: root/client/components/sidebar
diff options
context:
space:
mode:
authorMaxime Quandalle <maxime@quandalle.com>2015-09-16 15:35:18 +0200
committerMaxime Quandalle <maxime@quandalle.com>2015-09-16 15:35:18 +0200
commitec92f84f41dddf8826b5cf68e123a402599185e5 (patch)
tree00fae56365b6088daa67948ca91c3488f36b98b6 /client/components/sidebar
parent23218d0985f810cf233277fd45b7353a7c6327eb (diff)
parent6277bd2a90964a2ded6291153990bc0d26b56bae (diff)
downloadwekan-ec92f84f41dddf8826b5cf68e123a402599185e5.tar.gz
wekan-ec92f84f41dddf8826b5cf68e123a402599185e5.tar.bz2
wekan-ec92f84f41dddf8826b5cf68e123a402599185e5.zip
Merge pull request #305 from AlexanderS/fix/multiselect-toggle
Fix multiselect toggle logic This bug was introduced in 45b662a
Diffstat (limited to 'client/components/sidebar')
-rw-r--r--client/components/sidebar/sidebarFilters.js8
1 files changed, 4 insertions, 4 deletions
diff --git a/client/components/sidebar/sidebarFilters.js b/client/components/sidebar/sidebarFilters.js
index babd2f1e..bdecd63e 100644
--- a/client/components/sidebar/sidebarFilters.js
+++ b/client/components/sidebar/sidebarFilters.js
@@ -69,9 +69,9 @@ BlazeComponent.extendComponent({
const mappedSelection = this.mapSelection('label', labelId);
if (_.every(mappedSelection)) {
- mutateSelectedCards('addLabel', labelId);
- } else if (_.every(mappedSelection, (bool) => !bool)) {
mutateSelectedCards('removeLabel', labelId);
+ } else if (_.every(mappedSelection, (bool) => !bool)) {
+ mutateSelectedCards('addLabel', labelId);
} else {
const popup = Popup.open('disambiguateMultiLabel');
// XXX We need to have a better integration between the popup and the
@@ -83,9 +83,9 @@ BlazeComponent.extendComponent({
const memberId = this.currentData()._id;
const mappedSelection = this.mapSelection('member', memberId);
if (_.every(mappedSelection)) {
- mutateSelectedCards('assignMember', memberId);
- } else if (_.every(mappedSelection, (bool) => !bool)) {
mutateSelectedCards('unassignMember', memberId);
+ } else if (_.every(mappedSelection, (bool) => !bool)) {
+ mutateSelectedCards('assignMember', memberId);
} else {
const popup = Popup.open('disambiguateMultiMember');
// XXX We need to have a better integration between the popup and the