From bc59a63e5e0d57cb3ce119a963cbe40d41870582 Mon Sep 17 00:00:00 2001
From: Ray Schamp <ray@scratch.mit.edu>
Date: Thu, 6 Dec 2018 17:07:28 -0500
Subject: [PATCH 1/2] Only show the admin panel to admins

---
 src/views/preview/presentation.jsx | 32 +++++++++++++++++-------------
 src/views/preview/project-view.jsx |  1 +
 2 files changed, 19 insertions(+), 14 deletions(-)

diff --git a/src/views/preview/presentation.jsx b/src/views/preview/presentation.jsx
index 02e5e5c5a..643f98db9 100644
--- a/src/views/preview/presentation.jsx
+++ b/src/views/preview/presentation.jsx
@@ -110,6 +110,7 @@ const PreviewPresentation = ({
     remixes,
     replies,
     reportOpen,
+    showAdminPanel,
     showModInfo,
     singleCommentId,
     userOwnsProject,
@@ -180,22 +181,24 @@ const PreviewPresentation = ({
 
     return (
         <div className="preview">
-            <AdminPanel
-                className={classNames('project-admin-panel', {
-                    'admin-panel-open': adminPanelOpen,
-                    'modal-open': adminModalOpen
-                })}
-                isOpen={adminPanelOpen}
-                onClose={onCloseAdminPanel}
-                onOpen={onOpenAdminPanel}
-            >
-                <iframe
-                    className={classNames('admin-iframe', {
+            {showAdminPanel && (
+                <AdminPanel
+                    className={classNames('project-admin-panel', {
+                        'admin-panel-open': adminPanelOpen,
                         'modal-open': adminModalOpen
                     })}
-                    src={`/scratch2/${projectId}/adminpanel/`}
-                />
-            </AdminPanel>
+                    isOpen={adminPanelOpen}
+                    onClose={onCloseAdminPanel}
+                    onOpen={onOpenAdminPanel}
+                >
+                    <iframe
+                        className={classNames('admin-iframe', {
+                            'modal-open': adminModalOpen
+                        })}
+                        src={`/scratch2/${projectId}/adminpanel/`}
+                    />
+                </AdminPanel>
+            )}
             { projectInfo && projectInfo.author && projectInfo.author.id && (
                 <React.Fragment>
                     {banner}
@@ -646,6 +649,7 @@ PreviewPresentation.propTypes = {
     remixes: PropTypes.arrayOf(PropTypes.object),
     replies: PropTypes.objectOf(PropTypes.array),
     reportOpen: PropTypes.bool,
+    showAdminPanel: PropTypes.bool,
     showModInfo: PropTypes.bool,
     singleCommentId: PropTypes.oneOfType([PropTypes.number, PropTypes.bool]),
     userOwnsProject: PropTypes.bool,
diff --git a/src/views/preview/project-view.jsx b/src/views/preview/project-view.jsx
index 134ae836e..caee5d768 100644
--- a/src/views/preview/project-view.jsx
+++ b/src/views/preview/project-view.jsx
@@ -576,6 +576,7 @@ class Preview extends React.Component {
                         replies={this.props.replies}
                         reportOpen={this.state.reportOpen}
                         showModInfo={this.props.isAdmin}
+                        showAdminPanel={this.props.isAdmin}
                         singleCommentId={this.state.singleCommentId}
                         userOwnsProject={this.props.userOwnsProject}
                         visibilityInfo={this.props.visibilityInfo}

From 0957173c88adc52d299fab7c892f1638862206e4 Mon Sep 17 00:00:00 2001
From: Ray Schamp <ray@scratch.mit.edu>
Date: Thu, 6 Dec 2018 22:48:03 -0500
Subject: [PATCH 2/2] Fix CSS

I dunno what I was seeing that made me think the width should be 237, but it was incorrect.
---
 src/views/preview/preview.scss | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/views/preview/preview.scss b/src/views/preview/preview.scss
index 5fdbeef21..8090d47a9 100644
--- a/src/views/preview/preview.scss
+++ b/src/views/preview/preview.scss
@@ -43,7 +43,7 @@ $stage-width: 480px;
 
     .project-admin-panel.admin-panel-open {
         padding: 0;
-        width: 237px;
+        width: 250px;
     }
 
     .admin-iframe {
@@ -53,7 +53,7 @@ $stage-width: 480px;
         z-index: 100;
         margin: 0;
         border: 0;
-        width: 237px;
+        width: 250px;
         height: 100%;
     }