Sign Typed Request styling fixes

3 files changed, 18 insertions, 10 deletions

diff --git a/app/scripts/lib/notification-manager.js b/app/scripts/lib/notification-manager.js
index 98cd3f760..adaf60c65 100644
--- a/app/scripts/lib/notification-manager.js
+++ b/app/scripts/lib/notification-manager.js
@@ -1,5 +1,5 @@
 const extension = require('extensionizer')
-const height = 610
+const height = 620
 const width = 360
 
 
diff --git a/ui/app/components/signature-request.js b/ui/app/components/signature-request.js
index 35a739a8f..a0ecbe8ec 100644
--- a/ui/app/components/signature-request.js
+++ b/ui/app/components/signature-request.js
@@ -227,10 +227,10 @@ SignatureRequest.prototype.renderFooter = function () {
   }
 
   return h('div.request-signature__footer', [
-    h('div.request-signature__footer__cancel-button', {
+    h('button.request-signature__footer__cancel-button', {
       onClick: cancel,
     }, 'CANCEL'),
-    h('div.request-signature__footer__sign-button', {
+    h('button.request-signature__footer__sign-button', {
       onClick: sign,
     }, 'SIGN'),
   ])
diff --git a/ui/app/css/itcss/components/request-signature.scss b/ui/app/css/itcss/components/request-signature.scss
index e9ba7dbfd..ee54235d0 100644
--- a/ui/app/css/itcss/components/request-signature.scss
+++ b/ui/app/css/itcss/components/request-signature.scss
@@ -17,6 +17,10 @@
       top: 0;
       box-shadow: none;
     }
+
+    @media screen and (min-width: $break-large) {
+      max-height: 620px;
+    }
   }
 
   &__header {
@@ -27,6 +31,7 @@
     flex-flow: column;
     justify-content: center;
     align-items: center;
+    flex: 0 0 auto;
   }
 
   &__header-background {
@@ -105,6 +110,8 @@
     height: 100%;
     display: flex;
     flex-flow: column;
+    flex: 1 1 auto;
+    height: 0;
   }
 
   &__request-info {
@@ -142,13 +149,11 @@
     border-top: 1px solid $geyser;
     display: flex;
     flex-flow: column;
-    padding-right: 4px;
   }
 
   &__row {
     display: flex;
     flex-flow: column;
-    border-bottom: 1px solid $geyser;
   }
 
   &__row-title {
@@ -167,21 +172,21 @@
     font-family: Roboto;
     font-size: 14px;
     line-height: 19px;
-    margin-top: 6px;
-    margin-bottom: 15px;
-    margin-left: 18px;
-    width: 95%;
+    width: 100%;
     overflow-wrap: break-word;
+    border-bottom: 1px solid #d2d8dd;
+    padding: 6px 18px 15px;
   }
 
   &__footer {
-    height: 108px;
     width: 100%;
     display: flex;
     align-items: center;
     justify-content: space-evenly;
     font-size: 22px;
     position: relative;
+    flex: 0 0 auto;
+    border-top: 1px solid $geyser;
 
     &__cancel-button,
     &__sign-button {
@@ -198,17 +203,20 @@
       border-radius: 2px;
       box-shadow: none;
       max-width: 162px;
+      margin: 12px;
     }
 
     &__cancel-button {
       background: none;
       border: 1px solid $dusty-gray;
+      margin-right: 6px;
     }
 
     &__sign-button {
       background-color: $caribbean-green;
       border-width: 0;
       color: $white;
+      margin-left: 6px;
     }
   }
 }
\ No newline at end of file