summaryrefslogtreecommitdiffstats
path: root/client/components
diff options
context:
space:
mode:
authorAlexander Sulfrian <alexander@sulfrian.net>2015-09-16 04:30:02 +0200
committerAlexander Sulfrian <alexander@sulfrian.net>2015-09-16 04:30:02 +0200
commit6277bd2a90964a2ded6291153990bc0d26b56bae (patch)
treec75ba9c40b87c0e8b72f6ea195ce562bcf7a90a9 /client/components
parentbc1a576a670f4f7bcb447b399ce16cf7f38cde4f (diff)
downloadwekan-6277bd2a90964a2ded6291153990bc0d26b56bae.tar.gz
wekan-6277bd2a90964a2ded6291153990bc0d26b56bae.tar.bz2
wekan-6277bd2a90964a2ded6291153990bc0d26b56bae.zip
Fix multiselect toggle logic
If every element already has the label/member, we do not need to add it but it should be removed and if every element does not have the element, we should add it.
Diffstat (limited to 'client/components')
-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