added 100ms by queryselectorall(class) in changepizzaclasses

cameronwp · Sep 18, 2014 · 79dff93 · 79dff93
1 parent d7c4c56
commit 79dff93
Show file tree

Hide file tree

Showing 2 changed files with 25 additions and 21 deletions.
diff --git a/views/js/main.js b/views/js/main.js
@@ -357,9 +357,9 @@ var checkHeight = function(elem) {
 
 var pizzaElementGenerator = function(i) {
   var pizzaContainer = document.createElement("div");
-  pizzaContainer.classList.add("col-md-3");
+  pizzaContainer.classList.add("col-md-3","randomPizzaContainer");
   // pizzaContainer.style.height = "100%";
-  pizzaContainer.id = "pizza" + i.toString();
+  pizzaContainer.id = "pizza" + i;
 
   var pizzaImageContainer = document.createElement("div");
   pizzaImageContainer.classList.add("col-md-6");
@@ -375,7 +375,7 @@ var pizzaElementGenerator = function(i) {
   var pizzaDescriptionContainer = document.createElement("div");
   pizzaDescriptionContainer.classList.add("col-md-6");
   // pizzaDescriptionContainer.classList.add("col-md-push-3");
-  pizzaDescriptionContainer.id = "pizzaDescription";
+  // pizzaDescriptionContainer.id = "pizzaDescription";
 
   var pizzaName = document.createElement("h4");
   pizzaName.innerHTML = randomName();
@@ -451,7 +451,7 @@ var resizePizzas = function(size) {   // size is one of: "small", "medium", "lar
   // make this slower
   function changePizzaClasses (oldsize, newsize) {
 
-    for (var i = 0; i < 1000; i++) {
+    for (var i = 0; i < document.querySelectorAll(".randomPizzaContainer").length; i++) {
       var pizzas = document.querySelector("#randomPizzas");
       oldsize = findCurrentSize();
       pizzas.children[i].classList.remove(oldsize);
@@ -479,8 +479,8 @@ var resizePizzas = function(size) {   // size is one of: "small", "medium", "lar
 
 window.performance.mark("mark_start_generating"); // collect timing data
 
+var pizzasDiv = document.getElementById("randomPizzas");   // slows down initial generation by ~10ms
 for (var i = 2; i < 1000; i++) {
-  var pizzasDiv = document.getElementById("randomPizzas");   // slows down initial generation by ~10ms
   pizzasDiv.appendChild(pizzaElementGenerator(i));
 }
 
@@ -489,4 +489,8 @@ window.performance.measure("measure_pizza_generation", "mark_start_generating",
 var timeToGenerate = window.performance.getEntriesByName("measure_pizza_generation");
 console.log("Time to generate initial pizzas: " + timeToGenerate[0].duration + "ms");
 
-resizePizzas("xl");
+resizePizzas("xl");
+
+
+
+
diff --git a/views/pizza.html b/views/pizza.html
@@ -7,23 +7,23 @@
   <link rel="stylesheet" href="css/jquery-ui.structure.min.css">
   <link rel="stylesheet" href="css/jquery-ui.min.css">
 
-  <script src="js/jquery.js"></script>
-  <script src="js/jquery-ui.min.js"></script>
+  <!--<script src="js/jquery.js"></script>
+  <script src="js/jquery-ui.min.js"></script>-->
 
   <script>
-  $(function() {                // TODO: needs labels
-    $( "#slider" ).slider({
-      value:100,
-      min: 0,
-      max: 500,
-      step: 50,
-      slide: function( event, ui ) {
-        $( "#amount" ).val( "$" + ui.value );
-        console.log(ui.value);
-      }
-    });
-    $( "#amount" ).val( "$" + $( "#slider" ).slider( "value" ) );
-  });
+  // $(function() {                // TODO: needs labels
+  //   $( "#slider" ).slider({
+  //     value:100,
+  //     min: 0,
+  //     max: 500,
+  //     step: 50,
+  //     slide: function( event, ui ) {
+  //       $( "#amount" ).val( "$" + ui.value );
+  //       console.log(ui.value);
+  //     }
+  //   });
+  //   $( "#amount" ).val( "$" + $( "#slider" ).slider( "value" ) );
+  // });
   </script>
 
 </head>