Bugggg fixing

matthisk · matthisk · commit ee1e3d0dc6ca · 2017-05-02T14:48:48.000+02:00
diff --git a/src/components/Console/Console.js b/src/components/Console/Console.js
@@ -8,15 +8,17 @@ import JSConsole from './jsconsole'
 
 import './Console.scss'
 
+const CTRL_KEY = 'Ctrl';
 const HELP_TEXT = [
     '`Welcome to the interactive ES6 console',
     ' :help  -  Show this help text',
     '',
     ' Console commands:',
     ' Up & Down  -  Navigate command history',
     ' Enter      -  Execute code',
-    ' Ctrl-L     -  Clear console',
-    ' Ctrl-C     -  Cancel current command`',
+    ` ${CTRL_KEY}-L     -  Clear console`,
+    ` ${CTRL_KEY}-C     -  Cancel current command`,
+    ` ${CTRL_KEY}-S     -  Save snippet\``,
 
 ].join('\n');
 
diff --git a/src/components/Editor/Editor.js b/src/components/Editor/Editor.js
@@ -124,6 +124,7 @@ class _Editor extends Component {
         let {
             runCode,
             transformCode,
+            saveSnippet,
         } = this.props;
 
         let options = {
@@ -132,8 +133,12 @@ class _Editor extends Component {
             viewportMargin : Infinity,
             gutters : [GUTTER_ID,'CodeMirror-linenumbers'],
             extraKeys: {
-                'Cmd-Enter': cm => runCode(),
-                'Cmd-B': cm => transformCode(),
+                'Ctrl-Enter': cm => runCode(),
+                'Ctrl-B': cm => transformCode(),
+                'Ctrl-S': cm => {
+                    const code = cm.getValue();
+                    saveSnippet(code);
+                }
             }
         };
 
diff --git a/src/layouts/CoreLayout/CoreLayout.js b/src/layouts/CoreLayout/CoreLayout.js
@@ -174,6 +174,12 @@ class CoreLayout extends Component {
                 <Icon name='book' />
             </div>
 
+            <a className='sidebar-uno__bottom-two' href="//twitter.com/tthisk">
+                <div className='button-round'>
+                    <Icon name='twitter' />
+                </div>
+            </a>
+
             <a className='sidebar-uno__bottom' href="//github.com/matthisk/es6console">
                 <div className='button-round'>
                     <Icon name='github' />
diff --git a/src/layouts/CoreLayout/CoreLayout.scss b/src/layouts/CoreLayout/CoreLayout.scss
@@ -59,6 +59,11 @@
     }
 }
 
+.sidebar-uno__bottom-two {
+    @extend .sidebar-uno__bottom;
+    bottom: 45px;
+}
+
 .button-round {
     width: 34px;
     height: 34px;
diff --git a/src/routes/Home/components/HomeView.scss b/src/routes/Home/components/HomeView.scss
@@ -20,7 +20,7 @@
     padding-left: 25px;
     transition: flex .3s;
     min-height: 0px;
-    overflow-y: scroll;
+    overflow-y: hidden;
 }
 
 .ide__column-gutter {
diff --git a/src/store/ide.js b/src/store/ide.js
@@ -222,7 +222,7 @@ function transformOnTypeReducer(state, action) {
     let {
         errors = [],
         code = ''
-    } = transform(state, action);
+    } = transformReducer(state, action);
 
     let { editors } = state;
 
diff --git a/todo.md b/todo.md
@@ -5,28 +5,34 @@
   - show line numbers
   - select theme
   - indent unit
-[] Keyboard shortcuts
+[*] Keyboard shortcuts
   - [*] Cmd-Enter
   - [*] Cmd-B
-  - [] Cmd-S
+  - [*] Cmd-S
 [*] Console better formatting for log lines
 [*] Help text when you open console
 [*] Localstorage saving
 [*] Saving to psql
 [*] Loading from psql
 [*] Examples for es2016 and es2017 presets
+[*] Loading screen
 [] Draggable editor sizes
-[] Add Linter
-[] Loading screen
+[*] Add link to my Twitter
+
+###Ops
+[] Create test deploy to dokku
+[] Create real deploy to dokku
 
 ### Bugs
 
 [*] Console prints multiple times on run
 [] Catch errors on iframe (sandbox)
-[] Saving from a snippet page doesn't seem to work
+[*] Saving from a snippet page doesn't seem to work
 [] Async examples will not log all statements to the console
 [] CompletionValue is not logged to console
+[] locationChange should load snippet
 
 ### Future
 
+[] Add Linter
 [] Wiki

Original file line number	Diff line number	Diff line change
`@@ -59,6 +59,11 @@`
`59`	`59`	`}`
`60`	`60`	`}`
`61`	`61`
	`62`	`+.sidebar-uno__bottom-two {`
	`63`	`+ @extend .sidebar-uno__bottom;`
	`64`	`+ bottom: 45px;`
	`65`	`+}`
	`66`	`+`
`62`	`67`	`.button-round {`
`63`	`68`	`width: 34px;`
`64`	`69`	`height: 34px;`
Original file line number	Diff line number	Diff line change
`@@ -20,7 +20,7 @@`
`20`	`20`	`padding-left: 25px;`
`21`	`21`	`transition: flex .3s;`
`22`	`22`	`min-height: 0px;`
`23`		`- overflow-y: scroll;`
	`23`	`+ overflow-y: hidden;`
`24`	`24`	`}`
`25`	`25`
`26`	`26`	`.ide__column-gutter {`