Merge pull request #4333 from mfilser/card_details_list_select

Adding list select at card details

client/components/cards/cardDetails.jade

@@ -234,6 +234,15 @@ template(name="cardDetails")
                   +viewer
                     = sort
 
+        if currentBoard.allowsShowLists
+          .card-details-item.card-details-show-lists
+            h3.card-details-item-title
+              i.fa.fa-list
+              | {{_ 'list'}}
+            select.js-select-lists
+              each currentBoard.lists
+                option(value="{{_id}}" selected="{{#if isCurrentListId _id}}selected{{/if}}") {{title}}
+
         //.card-details-items
         if getSpentTime
           .card-details-item.card-details-item-spent

client/components/cards/cardDetails.js

@@ -168,6 +168,15 @@ BlazeComponent.extendComponent({
     );
   },
 
+  /** returns if the list id is the current list id
+   * @param listId list id to check
+   * @return is the list id the current list id ?
+   */
+  isCurrentListId(listId) {
+    const ret = this.data().listId == listId;
+    return ret;
+  },
+
   onRendered() {
     if (Meteor.settings.public.CARD_OPENED_WEBHOOK_ENABLED) {
       // Send Webhook but not create Activities records ---
@@ -379,6 +388,12 @@ BlazeComponent.extendComponent({
             card.move(card.boardId, card.swimlaneId, card.listId, sort);
           }
         },
+        'change .js-select-lists'(event) {
+          let card = this.data();
+          const listSelect = this.$('.js-select-lists')[0];
+          const listId = listSelect.options[listSelect.selectedIndex].value;
+          card.move(card.boardId, card.swimlaneId, listId, card.sort);
+        },
         'click .js-go-to-linked-card'() {
           Utils.goCardId(this.data().linkedId);
         },

client/components/cards/cardDetails.styl

@@ -211,13 +211,13 @@ avatar-radius = 50%
 
     .card-details-item
       margin-right: 0.5em
+      flex-grow: 1
       &:last-child
         margin-right: 0
       &.card-details-item-labels
         display: block
         word-wrap: break-word
         max-width: 95%
-        flex-grow: 1
       &.card-details-item-members,
       &.card-details-item-assignees,
       &.card-details-item-customfield,
@@ -225,7 +225,6 @@ avatar-radius = 50%
         display: block
         word-wrap: break-word
         max-width: 36%
-        flex-grow: 1
       &.card-details-item-creator,
       &.card-details-item-received,
       &.card-details-item-start,
@@ -234,7 +233,6 @@ avatar-radius = 50%
         display: block
         word-wrap: break-word
         max-width: 28%
-        flex-grow: 1
       &.custom-fields
         padding-left: 10px
 

client/components/sidebar/sidebar.jade

@@ -203,6 +203,12 @@ template(name="boardCardSettingsPopup")
         span
           i.fa.fa-sort
           | {{_ 'card-sorting-by-number'}}
+    div.check-div
+      a.flex.js-field-has-card-show-lists(class="{{#if allowsShowLists}}is-checked{{/if}}")
+        .materialCheckBox(class="{{#if allowsShowLists}}is-checked{{/if}}")
+        span
+          i.fa.fa-list
+          | {{_ 'card-show-lists'}}
     div.check-div
       a.flex.js-field-has-labels(class="{{#if allowsLabels}}is-checked{{/if}}")
         .materialCheckBox(class="{{#if allowsLabels}}is-checked{{/if}}")

client/components/sidebar/sidebar.js

@@ -827,6 +827,10 @@ BlazeComponent.extendComponent({
     return this.currentBoard.allowsCardSortingByNumber;
   },
 
+  allowsShowLists() {
+    return this.currentBoard.allowsShowLists;
+  },
+
   allowsLabels() {
     return this.currentBoard.allowsLabels;
   },
@@ -1061,6 +1065,22 @@ BlazeComponent.extendComponent({
             this.currentBoard.allowsCardSortingByNumber,
           );
         },
+        'click .js-field-has-card-show-lists'(evt) {
+          evt.preventDefault();
+          this.currentBoard.allowsShowLists = !this.currentBoard
+            .allowsShowLists;
+          this.currentBoard.setAllowsShowLists(
+            this.currentBoard.allowsShowLists,
+          );
+          $(`.js-field-has-card-show-lists ${MCB}`).toggleClass(
+            CKCLS,
+            this.currentBoard.allowsShowLists,
+          );
+          $('.js-field-has-card-show-lists').toggleClass(
+            CKCLS,
+            this.currentBoard.allowsShowLists,
+          );
+        },
         'click .js-field-has-labels'(evt) {
           evt.preventDefault();
           this.currentBoard.allowsLabels = !this.currentBoard.allowsLabels;

i18n/en.i18n.json

@@ -1144,5 +1144,6 @@
   "moveChecklistPopup-title": "Move Checklist",
   "newlineBecomesNewChecklistItem": "Newline becomes new checklist item",
   "copyChecklist": "Copy Checklist",
-  "copyChecklistPopup-title": "Copy Checklist"
+  "copyChecklistPopup-title": "Copy Checklist",
+  "card-show-lists": "Card Show Lists"
 }

models/boards.js

@@ -440,6 +440,14 @@ Boards.attachSchema(
       defaultValue: true,
     },
 
+    allowsShowLists: {
+      /**
+       * Does the board allows show lists on the card?
+       */
+      type: Boolean,
+      defaultValue: true,
+    },
+
     allowsAssignedBy: {
       /**
        * Does the board allows requested by?
@@ -1370,6 +1378,10 @@ Boards.mutations({
     return { $set: { allowsCardSortingByNumber } };
   },
 
+  setAllowsShowLists(allowsShowLists) {
+    return { $set: { allowsShowLists } };
+  },
+
   setAllowsAttachments(allowsAttachments) {
     return { $set: { allowsAttachments } };
   },

server/migrations.js

@@ -1103,3 +1103,19 @@ Migrations.add('assign-boardwise-card-numbers', () => {
     });
   })
 });
+
+Migrations.add('add-card-details-show-lists', () => {
+  Boards.update(
+    {
+      allowsShowLists: {
+        $exists: false,
+      },
+    },
+    {
+      $set: {
+        allowsShowLists: true,
+      },
+    },
+    noValidateMulti,
+  );
+});