[Lens] Fix Workspace hidden when using Safari (#92616)

dej611 · mbondyra · dej611 · commit e8f2f7e34110 · 2021-03-02T12:03:53.000+01:00
Co-authored-by: Marta Bondyra &lt;marta.bondyra@elastic.co&gt;
diff --git a/x-pack/plugins/lens/public/drag_drop/__snapshots__/drag_drop.test.tsx.snap b/x-pack/plugins/lens/public/drag_drop/__snapshots__/drag_drop.test.tsx.snap
diff --git a/x-pack/plugins/lens/public/drag_drop/drag_drop.scss b/x-pack/plugins/lens/public/drag_drop/drag_drop.scss
@@ -81,13 +81,6 @@
   }
 }
 
-.lnsDragDrop__container {
-  position: relative;
-  overflow: visible !important; // sass-lint:disable-line no-important
-  width: 100%;
-  height: 100%;
-}
-
 .lnsDragDrop__reorderableDrop {
   position: absolute;
   width: 100%;
diff --git a/x-pack/plugins/lens/public/drag_drop/drag_drop.tsx b/x-pack/plugins/lens/public/drag_drop/drag_drop.tsx
@@ -456,7 +456,7 @@ const DropInner = memo(function DropInner(props: DropInnerProps) {
     isActiveDropTarget && dropType !== 'reorder' && dragging?.ghost ? dragging.ghost : undefined;
 
   return (
-    <div className="lnsDragDrop__container">
+    <>
       {React.cloneElement(children, {
         'data-test-subj': dataTestSubj || 'lnsDragDrop',
         className: classNames(children.props.className, classes, className),
@@ -471,7 +471,7 @@ const DropInner = memo(function DropInner(props: DropInnerProps) {
             style: ghost.style,
           })
         : null}
-    </div>
+    </>
   );
 });
 
diff --git a/x-pack/plugins/lens/public/editor_frame_service/editor_frame/workspace_panel/workspace_panel_wrapper.scss b/x-pack/plugins/lens/public/editor_frame_service/editor_frame/workspace_panel/workspace_panel_wrapper.scss
@@ -11,6 +11,7 @@
   min-height: $euiSizeXXL * 10;
   overflow: visible;
   border: none;
+  height: 100%;
 
   .lnsWorkspacePanelWrapper__pageContentBody {
     @include euiScrollBar;

Original file line number	Diff line number	Diff line change
`@@ -81,13 +81,6 @@`
`81`	`81`	`}`
`82`	`82`	`}`
`83`	`83`
`84`		`-.lnsDragDrop__container {`
`85`		`- position: relative;`
`86`		`- overflow: visible !important; // sass-lint:disable-line no-important`
`87`		`- width: 100%;`
`88`		`- height: 100%;`
`89`		`-}`
`90`		`-`
`91`	`84`	`.lnsDragDrop__reorderableDrop {`
`92`	`85`	`position: absolute;`
`93`	`86`	`width: 100%;`