From 81bff4da357a87ad657ab02ef083d11845e68df9 Mon Sep 17 00:00:00 2001
From: Ben Halpern <bendhalpern@gmail.com>
Date: Thu, 29 Aug 2019 19:00:22 -0400
Subject: [PATCH] Fix internal mods interactivity and improve layout (#3879)

---
 app/controllers/internal/mods_controller.rb |  2 +-
 app/views/internal/mods/index.html.erb      |  1 +
 app/views/layouts/internal.html.erb         | 13 ++++++++++---
 3 files changed, 12 insertions(+), 4 deletions(-)

diff --git a/app/controllers/internal/mods_controller.rb b/app/controllers/internal/mods_controller.rb
index 05cc921a1..79460f0cb 100644
--- a/app/controllers/internal/mods_controller.rb
+++ b/app/controllers/internal/mods_controller.rb
@@ -15,6 +15,6 @@ class Internal::ModsController < Internal::ApplicationController
   def update
     @user = User.find(params[:id])
     AssignTagModerator.add_trusted_role(@user)
-    redirect_to "/internal/mods"
+    render body: nil # No response needed at the moment
   end
 end
diff --git a/app/views/internal/mods/index.html.erb b/app/views/internal/mods/index.html.erb
index 5a4a5f432..8dca7e054 100644
--- a/app/views/internal/mods/index.html.erb
+++ b/app/views/internal/mods/index.html.erb
@@ -69,6 +69,7 @@
 function submitForms() {
   var form = $(this);
   var valuesToSubmit = $(this).serialize();
+  form.parents(".wrapper").addClass("submitting-no-border")
   $.ajax({
     type: "POST",
     url: $(this).attr('action'), //sumbits it to the given url of the form
diff --git a/app/views/layouts/internal.html.erb b/app/views/layouts/internal.html.erb
index 1a38bed3b..a83c3f759 100644
--- a/app/views/layouts/internal.html.erb
+++ b/app/views/layouts/internal.html.erb
@@ -119,10 +119,18 @@
     border: 20px solid black;
   }
 
+  .submitting-no-border {
+    background: #96f2cc !important;
+  }
+
   .highlighted-no-border {
     background: #2effa8 !important;
   }
 
+  .highlighted-no-border input[type="submit"] {
+    display: none;
+  }
+
   .highlighted-border {
     border: 20px solid black;
   }
@@ -139,11 +147,10 @@
         <span class="icon-bar"></span>
         <span class="icon-bar"></span>
       </button>
-      <a class="navbar-brand" href="#">Internal DEV</a>
     </div>
     <div id="navbar" class="collapse navbar-collapse">
-      <ul class="nav navbar-nav">
-        <% controller_names = %w[comments articles users tags welcome broadcasts reports pages classified_listings tools chat_channels] %>
+      <ul class="nav navbar-nav" style="font-size: 0.97em;">
+        <% controller_names = %w[comments articles users tags welcome broadcasts reports pages classified_listings tools chat_channels growth mods] %>
         <% controller_names.each do |name| %>
           <li class="<%= "active" if controller_name == name %>"><a href="/internal/<%= name %>"><%= name %></a></li>
         <% end %>