fix(crud): menu not fully showing for field actions, remove old backg…

…rounds COMPASS-6186 (#3574)
mongodb-js · Oct 13, 2022 · 27e900f · 27e900f
1 parent 615e50e
commit 27e900f
Show file tree

Hide file tree

Showing 7 changed files with 5 additions and 57 deletions.
diff --git a/packages/compass-components/src/components/document-list/document-edit-actions-footer.tsx b/packages/compass-components/src/components/document-list/document-edit-actions-footer.tsx
@@ -193,6 +193,8 @@ const container = css({
   paddingLeft: spacing[3],
   alignItems: 'center',
   gap: spacing[2],
+  borderBottomLeftRadius: 'inherit',
+  borderBottomRightRadius: 'inherit',
 });
 
 const message = css({

diff --git a/packages/compass-components/src/components/keyline-card.tsx b/packages/compass-components/src/components/keyline-card.tsx
@@ -7,7 +7,6 @@ import { withTheme } from '../hooks/use-theme';
 const keylineStyles = css({
   border: `1px solid ${palette.gray.light2}`,
   borderRadius: spacing[2],
-  overflow: 'hidden',
 });
 
 const keylineLightThemeStyles = css({

diff --git a/packages/compass-crud/src/components/document-json-view.less b/packages/compass-crud/src/components/document-json-view.less
@@ -11,12 +11,11 @@
     margin-bottom: 5px;
 
     .json-document-is-editing {
-      box-shadow: 2px 5px 8px fade(@palette__black, 20%);
+      border-radius: inherit;
     }
 
     .json-ace-editor {
       padding: 10px 10px 10px 0;
-      background: @palette__white;
       overflow: auto;
       flex-basis: auto;
       flex-shrink: 1;

diff --git a/packages/compass-crud/src/components/document-list.less b/packages/compass-crud/src/components/document-list.less
@@ -5,17 +5,13 @@
   list-style: none;
   position: relative;
   width: 100%;
-  margin-bottom: 16px;
+  margin-bottom: 32px;
   padding-bottom: 16px;
 
   &-item {
     position: relative;
     margin-bottom: 5px;
 
-    .document {
-      background: @palette__white;
-    }
-
     &:last-child {
       margin-bottom: 0;
       padding-bottom: 16px;

diff --git a/packages/compass-crud/src/components/document.less b/packages/compass-crud/src/components/document.less
@@ -4,6 +4,7 @@
   position: relative;
   text-overflow: ellipsis;
   max-width: 100%;
+  border-radius: inherit;
 
   &::-webkit-scrollbar {
     display: none;
@@ -18,15 +19,6 @@
     position: relative;
   }
 
-  &-is-deleting {
-    box-shadow: 0 0 0 1px @palette__red--light-1,
-      2px 5px 8px fade(@palette__black, 20%);
-  }
-
-  &-is-editing {
-    box-shadow: 2px 5px 8px fade(@palette__black, 20%);
-  }
-
   &:hover {
     .document-actions-button {
       visibility: visible;

diff --git a/packages/compass-explain-plan/src/components/explain-summary/explain-summary.module.less b/packages/compass-explain-plan/src/components/explain-summary/explain-summary.module.less
@@ -1,7 +1,6 @@
 @import (reference) 'mongodb-compass/src/app/styles/index.less';
 
 .explain-summary {
-  background: @palette__white;
   padding: 12px;
 
   h3 {

diff --git a/packages/compass-schema/src/components/compass-schema/compass-schema.module.less b/packages/compass-schema/src/components/compass-schema/compass-schema.module.less
@@ -70,13 +70,9 @@
 
 :global {
   .schema-field-list {
-    // .list-group;
     .schema-field {
-      // .list-group-item;
       padding: 24px 0 0 0;
       margin-bottom: 8px;
-      background: @palette__white;
-      transition: all 250ms ease-out;
 
       .row {
         margin-bottom: 24px;
@@ -239,47 +235,12 @@
       background: @palette__white;
       border: 1px solid @palette__gray--light-1;
     }
-
-    // .schema-field-type-null {
-    //   background: @series-bg-1;
-    // }
-    // .schema-field-type-number {
-    //   background: @series-bg-2;
-    // }
-    // .schema-field-type-long {
-    //   background: @series-bg-3;
-    // }
-    // .schema-field-type-string {
-    //   background: @series-bg-4;
-    // }
-    // .schema-field-type-boolean {
-    //   background: @series-bg-5;
-    // }
-    // .schema-field-type-undefined {
-    //   background: @series-bg-6;
-    // }
-    // .schema-field-type-date {
-    //   background: @series-bg-7;
-    // }
-    // .schema-field-type-objectid {
-    //   background: @series-bg-8;
-    // }
-    // .schema-field-type-array {
-    //   background: @series-bg-1;
-    // }
-    // .schema-field-type-object {
-    //   background: @series-bg-1;
-    // }
   }
 }
 
 :global {
   .schema-value-list {
-    // TODO .small;
-    // TODO .list-unstyled;
-
     .schema-value-list-item {
-      // TODO .badge;
       display: inline-block;
       margin-left: 5px;
       margin-right: 5px;