From eeb1a80fa8a5b95e21464ccdb0be6c5458c3158c Mon Sep 17 00:00:00 2001
From: Nicolas Ronvel <gulix33xp@yahoo.fr>
Date: Tue, 7 Mar 2017 20:11:55 +0100
Subject: [PATCH] Correction on the Honorary Member

Filter changes were not added to the Honorary part
Better scroll on the Honorary Members list
---
 builder/css/styles.css                 |  4 +++-
 builder/index.html                     |  7 ++-----
 builder/js/viewmodels/teamBuilderVM.js |  1 +
 builder/js/viewmodels/teamVM.js        | 21 ++++++++++++++++++---
 4 files changed, 24 insertions(+), 9 deletions(-)

diff --git a/builder/css/styles.css b/builder/css/styles.css
index 840a48f..8763c56 100644
--- a/builder/css/styles.css
+++ b/builder/css/styles.css
@@ -183,7 +183,7 @@ h3 {
 .roster-supreme {
   width: 189px;
   height: 150px;
-  margin: 10px;
+  margin: 8px;
   -webkit-border-radius: 10px 10px 0 0;
   border-radius: 10px 10px 0 0;
   border-style: solid;
@@ -534,6 +534,8 @@ h3 {
 
 .modal-frame-large {
   overflow-y: scroll;
+  height: 100%;
+  width: 100%;
 }
 
 .modal-label {
diff --git a/builder/index.html b/builder/index.html
index 471354c..5bbee00 100644
--- a/builder/index.html
+++ b/builder/index.html
@@ -137,14 +137,11 @@ <h2>Recruit your Supremes</h2>
     </div>
     <!-- ko if: isHonoraryMembersSelectable -->
     <div class="modal-box">
-      <div class="modal-content">
+      <div class="modal-content" id="honorary-member-selection-box">
         <div class="icon-close modal-close" data-bind="click: endHonoraryMemberSelection"></div>
         <div class="modal-frame modal-frame-large">
           <div class="modal-label">Select your Honorary Member</div>
-          <div data-bind="with: $parent.supremeFilter">
-            <select data-bind="options: origins, optionsText: 'origin_label', value: selectedOrigin,"></select>
-            <select data-bind="options: roles, optionsText: 'role_label', value: selectedRole,"></select>
-          </div>
+          <supremes-filter params="filter: $parent.supremeFilter"></supremes-filter>
           <div class="recruitment-list scrollable-list" data-bind="foreach: possibleHonoraryMembers">
             <supreme-box params="supreme: $data"></supreme-box>
           </div>
diff --git a/builder/js/viewmodels/teamBuilderVM.js b/builder/js/viewmodels/teamBuilderVM.js
index b802fb7..a4d37fa 100644
--- a/builder/js/viewmodels/teamBuilderVM.js
+++ b/builder/js/viewmodels/teamBuilderVM.js
@@ -97,6 +97,7 @@ function teamBuilderVM()
   self.loadTeamForAffiliation = function(affiliationVM) {
     self.affiliations([]);
     self.team(TeamVM.newTeamVM(affiliationVM));
+    self.team().setBuilder(self);
     self.supremesPool(SupremeVM.loadForAffiliation(affiliationVM, self.recruitSupreme, self.dismissSupreme));
   }
 
diff --git a/builder/js/viewmodels/teamVM.js b/builder/js/viewmodels/teamVM.js
index 63adaef..8b42988 100644
--- a/builder/js/viewmodels/teamVM.js
+++ b/builder/js/viewmodels/teamVM.js
@@ -1,10 +1,12 @@
-define(['knockout',
+define(['jQuery',
+        'knockout',
         'lodash',
         'viewmodels/supremeVM',
         'scripts/teamCode',
         'scripts/url'
        ],
-function(ko,
+function($,
+         ko,
          _,
          SupremeVM,
          TeamCode,
@@ -31,6 +33,8 @@ function teamVM(affiliation)
   // Honorary Members
   self.isHonoraryMembersSelectable = ko.observable(false);
   self.supremeGrantingHonorary = ko.observable(null);
+  // TeamBuilder
+  self.teamBuilderVM = null;
 
   /**********************************/
   /* Accessors & Computed Variables */
@@ -116,7 +120,10 @@ function teamVM(affiliation)
     }
 
     // User Filter
-    //supremes = self.supremeFilter().filter(supremes);
+    if (self.teamBuilderVM != null) {
+      supremes = self.teamBuilderVM.supremeFilter().filter(supremes);
+    }
+
 
     return _.sortBy(supremes, [function(o) { return o.jsonData.name; }]);
   })
@@ -232,6 +239,10 @@ function teamVM(affiliation)
   /* Honorary Member */
   self.chooseHonoraryMember = function(supremeVM) {
     self.isHonoraryMembersSelectable(true);
+    // Height of the box
+    var viewportHeight = $(window).height();
+    $('#honorary-member-selection-box').css('height', viewportHeight * 0.8);
+        
     self.supremeGrantingHonorary(supremeVM);
   }
   self.endHonoraryMemberSelection = function() {
@@ -261,6 +272,10 @@ function teamVM(affiliation)
     self.dismissSupreme(supremeVM);
   }
 
+  self.setBuilder = function(teamBuilderVM) {
+    self.teamBuilderVM = teamBuilderVM;
+  }
+
   /*************************/
   /* Object Initialization */
   /*************************/