From be010b8ac0ac8411115cf4deb3af5eb107245b76 Mon Sep 17 00:00:00 2001
From: MathuraMG <mmg9291@gmail.com>
Date: Fri, 12 Aug 2016 14:19:23 -0400
Subject: [PATCH] switch div to sections

---
 client/modules/IDE/components/Editor.js       |  6 +++---
 client/modules/IDE/components/Preferences.js  |  2 +-
 client/modules/IDE/components/PreviewFrame.js | 20 +++++++++----------
 client/modules/IDE/components/Toolbar.js      |  4 ++--
 client/styles/abstracts/_placeholders.scss    |  1 +
 client/styles/components/_preferences.scss    |  2 +-
 static/intercept-p5.js                        |  5 +----
 7 files changed, 19 insertions(+), 21 deletions(-)
diff --git a/client/modules/IDE/components/Editor.js b/client/modules/IDE/components/Editor.js
index 640761f8..d8829736 100644
--- a/client/modules/IDE/components/Editor.js
+++ b/client/modules/IDE/components/Editor.js
@@ -99,14 +99,14 @@ class Editor extends React.Component {
 
   render() {
     return (
-      <div>
-        <div ref="container" className="editor-holder" tabIndex="0" title="code editor" role="main">
+      <section title="code editor" role="main">
+        <div ref="container" className="editor-holder" tabIndex="0">
         </div>
         <EditorAccessibility
           lintMessages={this.props.lintMessages}
           lineNo={this.props.lineNo}
         />
-      </div>
+      </section>
     );
   }
 }
diff --git a/client/modules/IDE/components/Preferences.js b/client/modules/IDE/components/Preferences.js
index c1102629..59f13c0a 100644
--- a/client/modules/IDE/components/Preferences.js
+++ b/client/modules/IDE/components/Preferences.js
@@ -154,7 +154,7 @@ class Preferences extends React.Component {
             >Off</button>
           </div>
         </div>
-        <div className="preference preference--hidden">
+        <div className="preference">
           <h4 className="preference__title">Lint Warning Sound</h4>
           <div className="preference__options">
             <button
diff --git a/client/modules/IDE/components/PreviewFrame.js b/client/modules/IDE/components/PreviewFrame.js
index 9e6abbf7..bfdfcb5f 100644
--- a/client/modules/IDE/components/PreviewFrame.js
+++ b/client/modules/IDE/components/PreviewFrame.js
@@ -135,7 +135,7 @@ class PreviewFrame extends React.Component {
     htmlHeadContents += '<script src="/data.js"></script>\n';
     htmlHeadContents += '<script src="/interceptor-functions.js"></script>\n';
     htmlHeadContents += '<script src="/intercept-p5.js"></script>\n';
-    htmlHeadContents += `<script type="text/javascript" src="http://chir.ag/projects/ntc/ntc.js"></script>`;
+    htmlHeadContents += '<script type="text/javascript" src="http://chir.ag/projects/ntc/ntc.js"></script>';
     htmlFile = htmlFile.replace(/(?:<head.*?>)([\s\S]*?)(?:<\/head>)/gmi, `<head>\n${htmlHeadContents}\n</head>`);
 
     // const htmlHead = htmlFile.match(/(?:<head.*?>)([\s\S]*?)(?:<\/head>)/gmi);
@@ -170,15 +170,15 @@ class PreviewFrame extends React.Component {
 
   render() {
     return (
-        <iframe
-          className="preview-frame"
-          aria-label="sketch output"
-          role="main"
-          tabIndex="0"
-          frameBorder="0"
-          title="sketch output"
-          sandbox="allow-scripts allow-pointer-lock allow-same-origin allow-popups allow-modals allow-forms"
-        />
+      <iframe
+        className="preview-frame"
+        aria-label="sketch output"
+        role="main"
+        tabIndex="0"
+        frameBorder="0"
+        title="sketch output"
+        sandbox="allow-scripts allow-pointer-lock allow-same-origin allow-popups allow-modals allow-forms"
+      />
     );
   }
 }
diff --git a/client/modules/IDE/components/Toolbar.js b/client/modules/IDE/components/Toolbar.js
index c69f04c5..9af73a23 100644
--- a/client/modules/IDE/components/Toolbar.js
+++ b/client/modules/IDE/components/Toolbar.js
@@ -21,7 +21,7 @@ function Toolbar(props) {
   });
 
   return (
-    <div className="toolbar">
+    <section className="toolbar" title="toolbar" role="main">
       <img className="toolbar__logo" src={logoUrl} alt="p5js Logo" />
       <button className={playButtonClass} onClick={props.startSketch} aria-label="play sketch">
         <InlineSVG src={playUrl} alt="Play Sketch" />
@@ -55,7 +55,7 @@ function Toolbar(props) {
       >
         <InlineSVG src={preferencesUrl} alt="Show Preferences" />
       </button>
-    </div>
+    </section>
   );
 }
 
diff --git a/client/styles/abstracts/_placeholders.scss b/client/styles/abstracts/_placeholders.scss
index 8149e3c6..a2a3bd02 100644
--- a/client/styles/abstracts/_placeholders.scss
+++ b/client/styles/abstracts/_placeholders.scss
@@ -71,6 +71,7 @@
 	color: $light-primary-text-color;
 	background-color: $light-modal-button-background-color;
 	padding: 0;
+	margin-bottom: #{20 / $base-font-size}rem;
 	line-height: #{50 / $base-font-size}rem;
 	& g {
 		fill: $light-primary-text-color;
diff --git a/client/styles/components/_preferences.scss b/client/styles/components/_preferences.scss
index 961e906e..1152a26a 100644
--- a/client/styles/components/_preferences.scss
+++ b/client/styles/components/_preferences.scss
@@ -35,7 +35,7 @@
 .preference {
 	display: flex;
 	flex-wrap: wrap;
-	padding-bottom: #{40 / $base-font-size}rem;
+	padding-bottom: #{12 / $base-font-size}rem;
 	& + & {
   	border-top: 2px dashed $light-button-border-color;
 	}
diff --git a/static/intercept-p5.js b/static/intercept-p5.js
index 0adc542b..eed538d7 100644
--- a/static/intercept-p5.js
+++ b/static/intercept-p5.js
@@ -1,9 +1,6 @@
 var shadowDOMElement;
 var canvasLocation ='';
 
-//for object in setpu (??)
-
-
 funcNames = refData["classitems"].map(function(x){
   return {
     name: x["name"],
@@ -15,7 +12,7 @@ funcNames = refData["classitems"].map(function(x){
 });
 
 funcNames = funcNames.filter(function(x) {
-  let className = x["class"];
+  var className = x["class"];
   return (x["name"] && x["params"] && (className==='p5'));
 })
 
