From 5c497d3eea7a0787b4af493219c6f4162730e151 Mon Sep 17 00:00:00 2001
From: Pascal Hartig <realpassy@fb.com>
Date: Thu, 20 Jun 2019 03:46:26 -0700
Subject: [PATCH] Make the sidebar pretty

Summary:
Now that the events tracking works again, make sure that the padding is not all over the place in the sidebar.

Before:

{F162525912}

Reviewed By: priteshrnandgaonkar

Differential Revision: D15875758

fbshipit-source-id: a30f057c8c0dc2dc77fc40f416ff565cc6621109
---
 src/plugins/fresco/ImagesSidebar.js | 13 +++----------
 src/ui/components/Panel.js          |  2 +-
 2 files changed, 4 insertions(+), 11 deletions(-)

diff --git a/src/plugins/fresco/ImagesSidebar.js b/src/plugins/fresco/ImagesSidebar.js
index ab69de78b..7611b3245 100644
--- a/src/plugins/fresco/ImagesSidebar.js
+++ b/src/plugins/fresco/ImagesSidebar.js
@@ -81,21 +81,14 @@ export default class ImagesSidebar extends Component<
 class EventDetails extends Component<{
   event: ImageEventWithId,
 }> {
-  static Container = styled(Panel)({
-    flexShrink: 0,
-    marginTop: '15px',
-  });
-
   render() {
     const {event} = this.props;
 
     return (
-      <EventDetails.Container
+      <Panel
         heading={<RequestHeader event={event} />}
         floating={false}
-        padded={false}
-        grow={false}
-        collapsed={false}>
+        padded={true}>
         <p>
           <DataDescriptionKey>Attribution</DataDescriptionKey>
           <span key="sep">: </span>
@@ -138,7 +131,7 @@ class EventDetails extends Component<{
           />
         </p>
         {this.renderViewportData()}
-      </EventDetails.Container>
+      </Panel>
     );
   }
 
diff --git a/src/ui/components/Panel.js b/src/ui/components/Panel.js
index 1b591296e..9a1014f47 100644
--- a/src/ui/components/Panel.js
+++ b/src/ui/components/Panel.js
@@ -140,7 +140,7 @@ export default class Panel extends React.Component<
         collapsed={collapsed}>
         <Panel.PanelHeader
           floating={floating}
-          padded={typeof heading === 'string'}
+          padded={padded || typeof heading === 'string'}
           onClick={this.onClick}>
           <span>
             {collapsable && (