From 8ffea70b2f2ae63ca5cc13e1ca267d4d46811955 Mon Sep 17 00:00:00 2001
From: Trevor Buckner <calculuschild@gmail.com>
Date: Fri, 27 Nov 2020 21:14:15 -0500
Subject: [PATCH] Remove metadata on the Homepage (#1121)

* Remove the metadata button on home page

1) Hopefully make it more clear that the Homepage is meant as a sandbox to test out the site, not a real brew that will be saved.

2) Also, avoid errors resulting from trying to modify the metadata of a brew that doesn't really exist yet.

* Lint
---
 client/homebrew/editor/editor.jsx             |  4 +++-
 .../homebrew/editor/snippetbar/snippetbar.jsx | 24 +++++++++++--------
 client/homebrew/pages/homePage/homePage.jsx   |  2 +-
 3 files changed, 18 insertions(+), 12 deletions(-)

diff --git a/client/homebrew/editor/editor.jsx b/client/homebrew/editor/editor.jsx
index 8925835f3..d85d36d34 100644
--- a/client/homebrew/editor/editor.jsx
+++ b/client/homebrew/editor/editor.jsx
@@ -23,6 +23,7 @@ const Editor = createClass({
 
 			metadata         : {},
 			onMetadataChange : ()=>{},
+			showMetaButton   : true
 		};
 	},
 	getInitialState : function() {
@@ -118,7 +119,8 @@ const Editor = createClass({
 					brew={this.props.value}
 					onInject={this.handleInject}
 					onToggle={this.handgleToggle}
-					showmeta={this.state.showMetadataEditor} />
+					showmeta={this.state.showMetadataEditor}
+					showMetaButton={this.props.showMetaButton} />
 				{this.renderMetadataEditor()}
 				<CodeEditor
 					ref='codeEditor'
diff --git a/client/homebrew/editor/snippetbar/snippetbar.jsx b/client/homebrew/editor/snippetbar/snippetbar.jsx
index cdbd9001a..18fcdb9af 100644
--- a/client/homebrew/editor/snippetbar/snippetbar.jsx
+++ b/client/homebrew/editor/snippetbar/snippetbar.jsx
@@ -12,15 +12,14 @@ const execute = function(val, brew){
 	return val;
 };
 
-
-
 const Snippetbar = createClass({
 	getDefaultProps : function() {
 		return {
-			brew     : '',
-			onInject : ()=>{},
-			onToggle : ()=>{},
-			showmeta : false
+			brew           : '',
+			onInject       : ()=>{},
+			onToggle       : ()=>{},
+			showmeta       : false,
+			showMetaButton : true
 		};
 	},
 
@@ -41,13 +40,18 @@ const Snippetbar = createClass({
 		});
 	},
 
+	renderMetadataButton : function(){
+		if(!this.props.showMetaButton) return;
+		return <div className={cx('toggleMeta', { selected: this.props.showmeta })}
+			onClick={this.props.onToggle}>
+			<i className='fa fa-bars' />
+		</div>;
+	},
+
 	render : function(){
 		return <div className='snippetBar'>
 			{this.renderSnippetGroups()}
-			<div className={cx('toggleMeta', { selected: this.props.showmeta })}
-				onClick={this.props.onToggle}>
-				<i className='fa fa-bars' />
-			</div>
+			{this.renderMetadataButton()}
 		</div>;
 	}
 });
diff --git a/client/homebrew/pages/homePage/homePage.jsx b/client/homebrew/pages/homePage/homePage.jsx
index 8795156e7..e41251d60 100644
--- a/client/homebrew/pages/homePage/homePage.jsx
+++ b/client/homebrew/pages/homePage/homePage.jsx
@@ -77,7 +77,7 @@ const HomePage = createClass({
 
 			<div className='content'>
 				<SplitPane onDragFinish={this.handleSplitMove} ref='pane'>
-					<Editor value={this.state.text} onChange={this.handleTextChange} ref='editor'/>
+					<Editor value={this.state.text} onChange={this.handleTextChange} showMetaButton={false} ref='editor'/>
 					<BrewRenderer text={this.state.text} />
 				</SplitPane>
 			</div>