Minor style changes (#31)

- Removed double margin from .table-details > .row
- Fixed panel borders

frontend/amundsen_application/static/js/components/TableDetail/styles.scss

@@ -1,53 +1,42 @@
 @import 'variables';
 
 .table-detail {
-  margin: 40px auto 48px;
-  display: flex;
-  flex-direction: row;
-}
-
-/*
-  white-space: nowrap breaks without this width on the row
-*/
-.table-detail .row {
-  width: 100%;
-  margin: 0;
-}
-
+  margin: 64px auto 48px;
 
-.error-label {
-  margin: 48px auto;
-  width: fit-content;
-}
+  .error-label {
+    margin: 48px auto;
+    width: fit-content;
+  }
 
-.detail-header {
-  margin-bottom: 32px;
-}
-.detail-header .title {
-  font-size: 32px;
-  overflow-wrap: break-word;
-  font-family: $font-family-sans-serif-bold;
-  font-weight: $font-weight-sans-serif-bold;
-}
+  .detail-header {
+    margin-bottom: 32px;
+  }
+  .detail-header .title {
+    font-size: 32px;
+    overflow-wrap: break-word;
+    font-family: $font-family-sans-serif-bold;
+    font-weight: $font-weight-sans-serif-bold;
+  }
 
-.detail-list-header > label {
-  font-size: 20px;
-  margin-top: auto;
-  margin-bottom: 32px;
-  width: fit-content;
-  line-height: 24px;
-  font-family: $font-family-sans-serif-bold;
-  font-weight: $font-weight-sans-serif-bold;
-}
+  .detail-list-header > label {
+    font-size: 20px;
+    margin-top: auto;
+    margin-bottom: 32px;
+    width: fit-content;
+    line-height: 24px;
+    font-family: $font-family-sans-serif-bold;
+    font-weight: $font-weight-sans-serif-bold;
+  }
 
-@media (min-width: $screen-md-min) {
+  @media (min-width: $screen-md-min) {
 
-  /* TODO: Delete this with Bootstrap V4 */
-  .float-md-right {
-    float: right;
-  }
+    /* TODO: Delete this with Bootstrap V4 */
+    .float-md-right {
+      float: right;
+    }
 
-  .detail-header {
-    margin-bottom: 72px;
+    .detail-header {
+      margin-bottom: 72px;
+    }
   }
 }

frontend/amundsen_application/static/js/components/common/EntityCard/styles.scss

@@ -1,25 +1,16 @@
 @import 'variables';
 
-
 .entity-card {
-  -webkit-box-shadow: 8px 9px 20px -15px rgba(0,0,0,0.46);
-  -moz-box-shadow: 8px 9px 20px -15px rgba(0,0,0,0.46);
-  box-shadow: 8px 9px 20px -15px rgba(0,0,0,0.46);
-  border: 1px solid $gray-lighter;
+  border-radius: 2px;
   border-top: 4px solid $gradient-4;
-  border-radius: 5px;
-  padding: 24px;
-  border-top-left-radius: 0px;
-  border-top-right-radius: 0px;
+  box-shadow: 0 0 24px -2px rgba(0, 0, 0, .2);
+  padding: 32px;
 }
 
-
 @media (max-width: $screen-sm-max) {
   .entity-card {
-    -webkit-box-shadow: none;
-    -moz-box-shadow: none;
-    box-shadow: none;
     border: 0;
+    box-shadow: none;
     padding: 0;
     margin-bottom: 32px;
   }

frontend/amundsen_application/static/js/components/common/Feedback/FeedbackForm/styles.scss

@@ -23,7 +23,7 @@
 }
 
 .radio-set .radio-set-item:not(:first-child) {
-  margin: 0px 0px 0px 14px;
+  margin-left: 12px;
 }
 
 .radio-set-item label {

frontend/amundsen_application/static/js/components/common/Feedback/styles.scss

@@ -1,36 +1,33 @@
 @import 'variables';
 
 .feedback-component {
+  box-shadow: 0 0 24px -2px rgba(0, 0, 0, .2);
+  bottom: 75px;
   position: fixed;
   right: 25px;
-  bottom: 75px;
   z-index: 2;
-  -webkit-box-shadow: 0 0 10px -1px rgba(0, 0, 0, .5);
-  -moz-box-shadow: 0 0 10px -1px rgba(0, 0, 0, .5);
-  box-shadow: 0 0 10px -1px rgba(0, 0, 0, .5);
 
   &.expanded {
     background-color: white;
-    padding: 24px;
-    width: 400px;
+    border-radius: 2px;
+    border-top: 4px solid $brand-primary;
     height: auto;
     min-height: 450px;
-    border: 1px solid $gray-lighter;
-    border-top: 4px solid $brand-primary;
-    border-radius: 0 0 5px 5px;
+    padding: 32px;
+    width: 400px;
   }
 
   &.collapsed {
+    align-items: center;
     background-color: $brand-primary;
     border: 2px solid white;
-    margin: 0;
-    height: 48px;
-    width: 48px;
-    border-radius: 100px; /* makes it a circle */
+    border-radius: 50%; /* makes it a circle */
+    cursor: pointer;
     display: flex;
+    height: 48px;
     justify-content: center;
-    align-items: center;
-    cursor: pointer;
+    margin: 0;
+    width: 48px;
 
     &:hover {
       opacity: 0.5;