Fix: Do not show preview sidebar if hasSidebar is false (#67)

Also do not show owner and uploader if those values are missing.

Author: priyajeet

src/components/ContentPreview/ContentPreview.js

@@ -88,7 +88,7 @@ class ContentPreview extends PureComponent<DefaultProps, Props, State> {
         locale: DEFAULT_PREVIEW_LOCALE,
         version: DEFAULT_PREVIEW_VERSION,
         hasSidebar: false,
-        showSidebar: false,
+        showSidebar: true,
         hasHeader: false,
         onLoad: noop,
         onNavigate: noop
@@ -417,7 +417,8 @@ class ContentPreview extends PureComponent<DefaultProps, Props, State> {
                     <Measure bounds onResize={this.onResize}>
                         {({ measureRef }) => <div ref={measureRef} className='bcpr-content' />}
                     </Measure>
-                    {showSidebar &&
+                    {hasSidebar &&
+                        showSidebar &&
                         <Sidebar
                             file={file}
                             getPreviewer={this.getPreviewer}

src/components/FileProperties/FileProperties.js

@@ -20,18 +20,22 @@ const FileProperties = ({ file, getLocalizedMessage }: Props) => {
     const { name: uploader = '' } = created_by || {};
     return (
         <dl>
-            <dt>
-                {getLocalizedMessage('buik.item.owner')}
-            </dt>
-            <dd>
-                {owner}
-            </dd>
-            <dt>
-                {getLocalizedMessage('buik.item.uploader')}
-            </dt>
-            <dd>
-                {uploader}
-            </dd>
+            {!!owner &&
+                <dt>
+                    {getLocalizedMessage('buik.item.owner')}
+                </dt>}
+            {!!owner &&
+                <dd>
+                    {owner}
+                </dd>}
+            {!!uploader &&
+                <dt>
+                    {getLocalizedMessage('buik.item.uploader')}
+                </dt>}
+            {!!uploader &&
+                <dd>
+                    {uploader}
+                </dd>}
             <dt>
                 {getLocalizedMessage('buik.item.created')}
             </dt>

src/components/Timeline/Timeline.scss

@@ -23,6 +23,7 @@
 
     .buik-timeline-label img {
         border-radius: 100%;
+        filter: brightness(1.5);
         height: 30px;
         vertical-align: middle;
         width: 30px;

test/preview.html

@@ -22,7 +22,7 @@
             window.onload = function() {
                 const { ContentExplorer } = Box;
                 const explorer = new ContentExplorer();
-                explorer.show('0', 'dcs5EcLt7cxE7IH7dpZ8af6zfwL1JKDE', {
+                explorer.show('0', 'YJVDFYupIRpy1osehfEvmd7YgQDu0Gmq', {
                     hasPreviewSidebar: true
                 });
             }