From a481668fea2d4e0e4d172c47d87f7a26ccf667b0 Mon Sep 17 00:00:00 2001
From: Ad Schellevis <ad@opnsense.org>
Date: Mon, 27 Feb 2017 20:19:10 +0100
Subject: [PATCH] (proxy) add a notice to inform the user about the need to
 download the list. closes https://github.com/opnsense/core/issues/1432

---
 .../mvc/app/views/OPNsense/Proxy/index.volt   | 31 ++++++++++++++-----
 1 file changed, 24 insertions(+), 7 deletions(-)

diff --git a/src/opnsense/mvc/app/views/OPNsense/Proxy/index.volt b/src/opnsense/mvc/app/views/OPNsense/Proxy/index.volt
index 2546e0f67..ecd99de78 100644
--- a/src/opnsense/mvc/app/views/OPNsense/Proxy/index.volt
+++ b/src/opnsense/mvc/app/views/OPNsense/Proxy/index.volt
@@ -56,6 +56,20 @@ POSSIBILITY OF SUCH DAMAGE.
                 }
         );
 
+        // when  closing DialogEditBlacklist, point the user to the download buttons
+        $("#DialogEditBlacklist").on("show.bs.modal", function () {
+            // wait some time before linking the save button, missing handle
+            setTimeout(function(){
+                $("#btn_DialogEditBlacklist_save").click(function(){
+                    $("#remoteACLchangeMessage").slideDown(1000, function(){
+                        setTimeout(function(){
+                            $("#remoteACLchangeMessage").slideUp(2000);
+                        }, 2000);
+                    });
+                });
+            }, 500);
+        });
+
         /**
          *
          * Reconfigure proxy - activate changes
@@ -253,17 +267,20 @@ POSSIBILITY OF SUCH DAMAGE.
                 </div>
                 </td>
                 <td>
-                <small class="hidden" for="help_for_proxy.forward.acl.remoteACLs.blacklist">
-                    {{ lang._('
-                    Add an item to the table to fetch a remote acl for blacklisting.%s
-                    You can enable or disable the blacklist list.%s
-                    The active blacklists will be merged with the settings under %sForward Proxy -> Access Control List%s.
-                    ') | format('<br/>','<br/>','<b>','</b>') }}
-                </small>
+                  <small class="hidden" for="help_for_proxy.forward.acl.remoteACLs.blacklist">
+                      {{ lang._('
+                      Add an item to the table to fetch a remote acl for blacklisting.%s
+                      You can enable or disable the blacklist list.%s
+                      The active blacklists will be merged with the settings under %sForward Proxy -> Access Control List%s.
+                      ') | format('<br/>','<br/>','<b>','</b>') }}
+                  </small>
                 </td>
             </tr>
             <tr>
                 <td colspan="2">
+                    <div id="remoteACLchangeMessage" class="alert alert-info" style="display: none" role="alert">
+                        {{ lang._('Note: after changing categories, please remember to download the ACL again to apply your new settings') }}
+                    </div>
                     <table id="grid-remote-blacklists" class="table table-condensed table-hover table-striped table-responsive" data-editDialog="DialogEditBlacklist">
                         <thead>
                         <tr>