From bb06735c833f8d9bea021dd24ecc0826e6f61a72 Mon Sep 17 00:00:00 2001
From: dandds <dan-ctr@friends.dds.mil>
Date: Wed, 1 May 2019 06:24:15 -0400
Subject: [PATCH] updates to styles on app settings page:

- global: more bottom padding for panels
- app description textarea has no width contraint, constrained heigh
- fix extra padding in panel footers
- larger form columns for app name and description
- remove deprecated environment list
- do not need extra right padding on nested list items
---
 styles/components/_accordion_table.scss       |  2 +-
 styles/components/_portfolio_layout.scss      | 10 +++++++
 styles/elements/_panels.scss                  |  4 +--
 .../portfolios/applications/settings.html     | 30 ++++---------------
 4 files changed, 19 insertions(+), 27 deletions(-)

diff --git a/styles/components/_accordion_table.scss b/styles/components/_accordion_table.scss
index 119f1c31..ecb37d83 100644
--- a/styles/components/_accordion_table.scss
+++ b/styles/components/_accordion_table.scss
@@ -110,7 +110,7 @@
       font-size: $small-font-size;
       font-weight: $font-normal;
       background-color: $color-gray-lightest;
-      padding: $gap*2 $gap*7 $gap*5 $gap*4;
+      padding: $gap*2 $gap*4 $gap*5 $gap*4;
 
       th, td {
         .icon-link {
diff --git a/styles/components/_portfolio_layout.scss b/styles/components/_portfolio_layout.scss
index 335d4e84..5258b54b 100644
--- a/styles/components/_portfolio_layout.scss
+++ b/styles/components/_portfolio_layout.scss
@@ -336,6 +336,16 @@
         }
       }
     }
+
+    textarea {
+      max-height: 9rem;
+      max-width: none;
+    }
+
+    .panel__footer {
+      padding-bottom: 0;
+      padding-top: 0;
+    }
   }
 
   .activity-log {
diff --git a/styles/elements/_panels.scss b/styles/elements/_panels.scss
index 0036e474..96bb04dd 100644
--- a/styles/elements/_panels.scss
+++ b/styles/elements/_panels.scss
@@ -22,10 +22,10 @@
   margin-top: 0;
   margin-left: 0;
   margin-right: 0;
-  margin-bottom: $site-margins-mobile * 2;
+  margin-bottom: $site-margins-mobile * 6;
 
   @include media($medium-screen) {
-    margin-bottom: $site-margins * 2;
+    margin-bottom: $site-margins * 8;
   }
 }
 
diff --git a/templates/portfolios/applications/settings.html b/templates/portfolios/applications/settings.html
index 36717346..6570ccc1 100644
--- a/templates/portfolios/applications/settings.html
+++ b/templates/portfolios/applications/settings.html
@@ -20,10 +20,10 @@
           {{ "fragments.edit_application_form.explain" | translate }}
         </p>
         <div class="form-row">
-          <div class="form-col form-col--half">
+          <div class="form-col form-col--two-thirds">
             {{ TextInput(form.name) }}
           </div>
-          <div class="form-col form-col--half">
+          <div class="form-col form-col--third">
             {% if user_can(permissions.DELETE_APPLICATION) %}
             <div class="usa-input">
               <label for="delete-application">
@@ -43,31 +43,13 @@
           </div>
         </div>
         <div class="form-row">
-          <div class="form-col form-col--half">
+          <div class="form-col form-col--two-thirds">
             {{ TextInput(form.description, paragraph=True) }}
           </div>
+          <div class="form-col form-col--third">
+            &nbsp;
+          </div>
         </div>
-
-        <div class="application-list-item">
-          <header>
-            <h2 class="block-list__title">{{ 'portfolios.applications.environments_heading' | translate }}</h2>
-            <p>
-              {{ 'portfolios.applications.environments_description' | translate }}
-            </p>
-          </header>
-
-          <ul>
-            {% for environment in application.environments %}
-              <li class="application-edit__env-list-item">
-                <div class="usa-input input--disabled">
-                  <label>Environment Name</label>
-                  <input type="text" disabled value="{{ environment.name }}" readonly />
-                </div>
-              </li>
-            {% endfor %}
-          </ul>
-        </div>
-
       </div>
       <div class="panel__footer">
         <div class="action-group">