From 6bd6555554966d5d2f06328131edd1843e9d831b Mon Sep 17 00:00:00 2001
From: Amauri CHAMPEAUX <amauri@ferank.fr>
Date: Tue, 11 Sep 2018 10:40:16 +0200
Subject: [PATCH 1/3] Better scroll management

---
 css/tarteaucitron.css | 24 ++++++++++++++++++++++++
 1 file changed, 24 insertions(+)

diff --git a/css/tarteaucitron.css b/css/tarteaucitron.css
index ab51810..8e68326 100644
--- a/css/tarteaucitron.css
+++ b/css/tarteaucitron.css
@@ -42,6 +42,30 @@
     /*outline: 2px solid #cb3333;*/
 }
 
+/***
+ * Better scroll management
+ */
+div#tarteaucitronMainLineOffset {
+    margin-top: 0!important;
+}
+
+div#tarteaucitronServices {
+    margin-top: 21px!important;
+}
+
+#tarteaucitronServices::-webkit-scrollbar {
+    width: 5px;
+}
+ 
+#tarteaucitronServices::-webkit-scrollbar-track {
+    -webkit-box-shadow: inset 0 0 0 rgba(0,0,0,0);
+}
+ 
+#tarteaucitronServices::-webkit-scrollbar-thumb {
+  background-color: #ddd;
+  outline: 0px solid slategrey;
+}
+
 /***
  * Responsive layout for the control panel
  */

From e1565385557201c1186393faa979d7c40e032abe Mon Sep 17 00:00:00 2001
From: Amauri CHAMPEAUX <amauri@ferank.fr>
Date: Tue, 11 Sep 2018 10:40:49 +0200
Subject: [PATCH 2/3] Better scroll management

---
 tarteaucitron.js | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)

diff --git a/tarteaucitron.js b/tarteaucitron.js
index f5503ad..1af4756 100644
--- a/tarteaucitron.js
+++ b/tarteaucitron.js
@@ -16,7 +16,7 @@ var scripts = document.getElementsByTagName('script'),
 
 
 var tarteaucitron = {
-    "version": 20180910,
+    "version": 20180911002,
     "cdn": cdn,
     "user": {},
     "lang": {},
@@ -289,7 +289,7 @@ var tarteaucitron = {
                 html += '            </button>';
                 html += '         </div>';
                 html += '      </div>';
-                html += '      <div class="tarteaucitronBorder" id="tarteaucitronScrollbarParent">';
+                html += '      <div class="tarteaucitronBorder">';
                 html += '         <div class="clear"></div><ul>';
                 for (i = 0; i < cat.length; i += 1) {
                     html += '         <li id="tarteaucitronServicesTitle_' + cat[i] + '" class="tarteaucitronHidden">';
@@ -932,20 +932,20 @@ var tarteaucitron = {
                 if (document.getElementById('tarteaucitron') !== null && document.getElementById('tarteaucitronClosePanel') !== null && document.getElementById('tarteaucitronMainLineOffset') !== null) {
 
                     // reset
-                    tarteaucitron.userInterface.css('tarteaucitronScrollbarParent', 'height', 'auto');
+                    tarteaucitron.userInterface.css('tarteaucitronServices', 'height', 'auto');
 
                     // calculate
                     mainHeight = document.getElementById('tarteaucitron').offsetHeight;
                     closeButtonHeight = document.getElementById('tarteaucitronClosePanel').offsetHeight;
-                    headerHeight = document.getElementById('tarteaucitronMainLineOffset').offsetHeight;
 
                     // apply
-                    servicesHeight = (mainHeight - closeButtonHeight - headerHeight + 1);
-                    tarteaucitron.userInterface.css('tarteaucitronScrollbarParent', 'height', servicesHeight + 'px');
+                    servicesHeight = (mainHeight - closeButtonHeight + 2);
+                    tarteaucitron.userInterface.css('tarteaucitronServices', 'height', servicesHeight + 'px');
+                    tarteaucitron.userInterface.css('tarteaucitronServices', 'overflow-x', 'auto');
                 }
 
                 // align the main allow/deny button depending on scrollbar width
-                if (document.getElementById('tarteaucitronScrollbarParent') !== null && document.getElementById('tarteaucitronScrollbarChild') !== null) {
+                if (document.getElementById('tarteaucitronServices') !== null && document.getElementById('tarteaucitronScrollbarChild') !== null) {
 
                     // media query
                     if (e[a + 'Width'] <= 479) {
@@ -954,7 +954,7 @@ var tarteaucitron = {
                         scrollbarMarginRight = 12;
                     }
 
-                    scrollbarWidthParent = document.getElementById('tarteaucitronScrollbarParent').offsetWidth;
+                    scrollbarWidthParent = document.getElementById('tarteaucitronServices').offsetWidth;
                     scrollbarWidthChild = document.getElementById('tarteaucitronScrollbarChild').offsetWidth;
                     tarteaucitron.userInterface.css('tarteaucitronScrollbarAdjust', 'marginRight', ((scrollbarWidthParent - scrollbarWidthChild) + scrollbarMarginRight) + 'px');
                 }

From 63f0ba6866e16cbba1f1ffdb60c686464603c6ca Mon Sep 17 00:00:00 2001
From: Amauri CHAMPEAUX <amauri@ferank.fr>
Date: Tue, 11 Sep 2018 10:52:18 +0200
Subject: [PATCH 3/3] Restore effect on general allow/deny buttons

---
 tarteaucitron.js | 14 +++++++++-----
 1 file changed, 9 insertions(+), 5 deletions(-)

diff --git a/tarteaucitron.js b/tarteaucitron.js
index 1af4756..ec2420c 100644
--- a/tarteaucitron.js
+++ b/tarteaucitron.js
@@ -696,13 +696,15 @@ var tarteaucitron = {
 
             if (nbDenied === 0 && nbPending === 0) {
                 tarteaucitron.userInterface.css(c + 'AllAllowed', 'backgroundColor', greenDark);
-                tarteaucitron.userInterface.css(c + 'AllDenied', 'backgroundColor', gray);
+                tarteaucitron.userInterface.css(c + 'AllDenied', 'opacity', '0.4');
+                tarteaucitron.userInterface.css(c + 'AllAllowed', 'opacity', '1');
             } else if (nbAllowed === 0 && nbPending === 0) {
-                tarteaucitron.userInterface.css(c + 'AllAllowed', 'backgroundColor', gray);
+                tarteaucitron.userInterface.css(c + 'AllAllowed', 'opacity', '0.4');
+                tarteaucitron.userInterface.css(c + 'AllDenied', 'opacity', '1');
                 tarteaucitron.userInterface.css(c + 'AllDenied', 'backgroundColor', redDark);
             } else {
-                tarteaucitron.userInterface.css(c + 'AllAllowed', 'backgroundColor', gray);
-                tarteaucitron.userInterface.css(c + 'AllDenied', 'backgroundColor', gray);
+                tarteaucitron.userInterface.css(c + 'AllAllowed', 'opacity', '1');
+                tarteaucitron.userInterface.css(c + 'AllDenied', 'opacity', '1');
             }
 
             // close the alert if all service have been reviewed
@@ -877,7 +879,9 @@ var tarteaucitron = {
             allDivs = main.childNodes;
 
             if (typeof Array.prototype.map === 'function') {
-                Array.prototype.map.call(main.children, Object).sort(function (a, b) {
+                //Array.prototype.map.call(main.children, Object).sort(function (a, b) {
+                var mainChildren = Array.from(main.children);
+	            mainChildren.sort(function (a, b) {
                     if (tarteaucitron.services[a.id.replace(/Line/g, '')].name > tarteaucitron.services[b.id.replace(/Line/g, '')].name) { return 1; }
                     if (tarteaucitron.services[a.id.replace(/Line/g, '')].name < tarteaucitron.services[b.id.replace(/Line/g, '')].name) { return -1; }
                     return 0;