From 93cfd52e36d7982201fe1baa25e9bb0b7a43633d Mon Sep 17 00:00:00 2001
From: Tanuj Mittal <tamitta@microsoft.com>
Date: Fri, 5 Feb 2021 14:56:50 +0530
Subject: [PATCH] Update Code of Conduct Overlay and other minor changes (#424)

* Use light theme for coc-overlay

* Updates

* Fix vertical height for COC overlay
---
 .../Controls/NotebookGallery/CodeOfConductComponent.tsx    | 2 +-
 .../Controls/NotebookGallery/GalleryViewerComponent.less   | 2 ++
 .../Controls/NotebookGallery/GalleryViewerComponent.tsx    | 7 +------
 .../__snapshots__/CodeOfConductComponent.test.tsx.snap     | 2 +-
 4 files changed, 5 insertions(+), 8 deletions(-)

diff --git a/src/Explorer/Controls/NotebookGallery/CodeOfConductComponent.tsx b/src/Explorer/Controls/NotebookGallery/CodeOfConductComponent.tsx
index f2b2d63ec..9705dbfaa 100644
--- a/src/Explorer/Controls/NotebookGallery/CodeOfConductComponent.tsx
+++ b/src/Explorer/Controls/NotebookGallery/CodeOfConductComponent.tsx
@@ -101,7 +101,7 @@ export class CodeOfConductComponent extends React.Component<CodeOfConductCompone
                 fontSize: 12,
               },
             }}
-            label="I have read and accepted the code of conduct."
+            label="I have read and accept the code of conduct."
             onChange={this.onChangeCheckbox}
           />
         </Stack.Item>
diff --git a/src/Explorer/Controls/NotebookGallery/GalleryViewerComponent.less b/src/Explorer/Controls/NotebookGallery/GalleryViewerComponent.less
index 60e3180fb..ae87fe42c 100644
--- a/src/Explorer/Controls/NotebookGallery/GalleryViewerComponent.less
+++ b/src/Explorer/Controls/NotebookGallery/GalleryViewerComponent.less
@@ -10,9 +10,11 @@
 
 .publicGalleryTabContainer {
   position: relative;
+  height: 100vh;
 }
 
 .publicGalleryTabOverlayContent {
   background: white;
   padding: 20px;
+  margin: 10%;
 }
diff --git a/src/Explorer/Controls/NotebookGallery/GalleryViewerComponent.tsx b/src/Explorer/Controls/NotebookGallery/GalleryViewerComponent.tsx
index f4035b9ad..58715a5cc 100644
--- a/src/Explorer/Controls/NotebookGallery/GalleryViewerComponent.tsx
+++ b/src/Explorer/Controls/NotebookGallery/GalleryViewerComponent.tsx
@@ -164,12 +164,7 @@ export class GalleryViewerComponent extends React.Component<GalleryViewerCompone
 
     if (this.props.container?.isGalleryPublishEnabled()) {
       tabs.push(this.createFavoritesTab(GalleryTab.Favorites, this.state.favoriteNotebooks));
-
-      // explicitly checking if isCodeOfConductAccepted is not false, as it is initially undefined.
-      // Displaying code of conduct component on gallery load should not be the default behavior.
-      if (this.state.isCodeOfConductAccepted !== false) {
-        tabs.push(this.createPublishedNotebooksTab(GalleryTab.Published, this.state.publishedNotebooks));
-      }
+      tabs.push(this.createPublishedNotebooksTab(GalleryTab.Published, this.state.publishedNotebooks));
     }
 
     const pivotProps: IPivotProps = {
diff --git a/src/Explorer/Controls/NotebookGallery/__snapshots__/CodeOfConductComponent.test.tsx.snap b/src/Explorer/Controls/NotebookGallery/__snapshots__/CodeOfConductComponent.test.tsx.snap
index 5b1f48bb1..e080d5797 100644
--- a/src/Explorer/Controls/NotebookGallery/__snapshots__/CodeOfConductComponent.test.tsx.snap
+++ b/src/Explorer/Controls/NotebookGallery/__snapshots__/CodeOfConductComponent.test.tsx.snap
@@ -38,7 +38,7 @@ exports[`CodeOfConductComponent renders 1`] = `
   </StackItem>
   <StackItem>
     <StyledCheckboxBase
-      label="I have read and accepted the code of conduct."
+      label="I have read and accept the code of conduct."
       onChange={[Function]}
       styles={
         Object {