From 236948a2d2c5c65d5bb257f24933a64e1d8a05f1 Mon Sep 17 00:00:00 2001
From: Florrie <towerofnix@gmail.com>
Date: Thu, 6 Dec 2018 22:29:18 -0400
Subject: [PATCH 1/2] Treat defaultValue from Blockly prompts properly

That is, as the default value for the input, not the placeholder.
---
 src/components/prompt/prompt.jsx | 6 ++++--
 src/containers/blocks.jsx        | 2 +-
 src/containers/prompt.jsx        | 8 ++++++--
 3 files changed, 11 insertions(+), 5 deletions(-)

diff --git a/src/components/prompt/prompt.jsx b/src/components/prompt/prompt.jsx
index 78192f3ad..47b52219a 100644
--- a/src/components/prompt/prompt.jsx
+++ b/src/components/prompt/prompt.jsx
@@ -48,9 +48,10 @@ const PromptComponent = props => (
                 <input
                     autoFocus
                     className={styles.variableNameTextInput}
+                    defaultValue={props.defaultValue}
                     name={props.label}
-                    placeholder={props.placeholder}
                     onChange={props.onChange}
+                    onFocus={props.onFocus}
                     onKeyPress={props.onKeyPress}
                 />
             </Box>
@@ -133,14 +134,15 @@ const PromptComponent = props => (
 );
 
 PromptComponent.propTypes = {
+    defaultValue: PropTypes.string,
     isStage: PropTypes.bool.isRequired,
     label: PropTypes.string.isRequired,
     onCancel: PropTypes.func.isRequired,
     onChange: PropTypes.func.isRequired,
+    onFocus: PropTypes.func.isRequired,
     onKeyPress: PropTypes.func.isRequired,
     onOk: PropTypes.func.isRequired,
     onOptionSelection: PropTypes.func.isRequired,
-    placeholder: PropTypes.string,
     showMoreOptions: PropTypes.bool.isRequired,
     title: PropTypes.string.isRequired
 };
diff --git a/src/containers/blocks.jsx b/src/containers/blocks.jsx
index 9a8e5c9b6..ee002d3a6 100644
--- a/src/containers/blocks.jsx
+++ b/src/containers/blocks.jsx
@@ -450,9 +450,9 @@ class Blocks extends React.Component {
                 />
                 {this.state.prompt ? (
                     <Prompt
+                        defaultValue={this.state.prompt.defaultValue}
                         isStage={vm.runtime.getEditingTarget().isStage}
                         label={this.state.prompt.message}
-                        placeholder={this.state.prompt.defaultValue}
                         showMoreOptions={this.state.prompt.showMoreOptions}
                         title={this.state.prompt.title}
                         onCancel={this.handlePromptClose}
diff --git a/src/containers/prompt.jsx b/src/containers/prompt.jsx
index 72c8ad8a3..6161a3ea6 100644
--- a/src/containers/prompt.jsx
+++ b/src/containers/prompt.jsx
@@ -21,6 +21,9 @@ class Prompt extends React.Component {
     handleKeyPress (event) {
         if (event.key === 'Enter') this.handleOk();
     }
+    handleFocus (event) {
+        event.target.select();
+    }
     handleOk () {
         this.props.onOk(this.state.inputValue, this.state.optionSelection);
     }
@@ -36,13 +39,14 @@ class Prompt extends React.Component {
     render () {
         return (
             <PromptComponent
+                defaultValue={this.props.defaultValue}
                 isStage={this.props.isStage}
                 label={this.props.label}
-                placeholder={this.props.placeholder}
                 showMoreOptions={this.props.showMoreOptions}
                 title={this.props.title}
                 onCancel={this.handleCancel}
                 onChange={this.handleChange}
+                onFocus={this.handleFocus}
                 onKeyPress={this.handleKeyPress}
                 onOk={this.handleOk}
                 onOptionSelection={this.handleOptionSelection}
@@ -52,11 +56,11 @@ class Prompt extends React.Component {
 }
 
 Prompt.propTypes = {
+    defaultValue: PropTypes.string,
     isStage: PropTypes.bool.isRequired,
     label: PropTypes.string.isRequired,
     onCancel: PropTypes.func.isRequired,
     onOk: PropTypes.func.isRequired,
-    placeholder: PropTypes.string,
     showMoreOptions: PropTypes.bool.isRequired,
     title: PropTypes.string.isRequired
 };

From aab8a3bab6d5483499c95a658b2813ae4d16dcaf Mon Sep 17 00:00:00 2001
From: Karishma Chadha <kchadha@media.mit.edu>
Date: Tue, 11 Dec 2018 20:45:47 -0400
Subject: [PATCH 2/2] Fix mistake from merge commit

Co-Authored-By: towerofnix <9948030+towerofnix@users.noreply.github.com>
---
 src/containers/blocks.jsx | 1 -
 1 file changed, 1 deletion(-)

diff --git a/src/containers/blocks.jsx b/src/containers/blocks.jsx
index 08cb7953a..80cc87167 100644
--- a/src/containers/blocks.jsx
+++ b/src/containers/blocks.jsx
@@ -464,7 +464,6 @@ class Blocks extends React.Component {
                         isStage={vm.runtime.getEditingTarget().isStage}
                         label={this.state.prompt.message}
                         showCloudOption={this.state.prompt.showCloudOption}
-                        showMoreOptions={this.state.prompt.showMoreOptions}
                         showVariableOptions={this.state.prompt.showVariableOptions}
                         title={this.state.prompt.title}
                         vm={vm}