Topic1 by Facundo97Maldonado · Pull Request #2 · Facundo97Maldonado/startup

Facundo97Maldonado · 2017-04-26T18:13:17Z

No description provided.

This reverts commit 1353d7f.

sebadlf · 2017-04-27T18:04:50Z

01-JS-DOM-APIs/css/estilosX.css

@@ -0,0 +1,28 @@
+.x1{


Estos estilos no los veo aplicados en ningún lado

Estos nombres de clases son horripilantes xD

sebadlf · 2017-04-27T18:05:47Z

01-JS-DOM-APIs/index.html

+	<link rel="stylesheet" type="text/css" href="myStylesheet.css">
+</head>
+<body>
+<script src="https://ajax.googleapis.com/ajax/libs/angularjs/1.4.8/angular.min.js"></script>


Angular? estimo que esto esta de más

sebadlf · 2017-04-27T18:08:15Z

01-JS-DOM-APIs/index.html

+<!--JAVASCRIPT -->
+<!-- Function to change the text of a button when you drag the mouse over him -->
+<script>
+	function changeText(id){


Es solo un tema de nombre id es en realidad el elemento del DOM que representa el botton. Probablemente "target" o "element" serian nombre mas adecuados

sebadlf · 2017-04-27T18:19:18Z

01-JS-DOM-APIs/index.html

+	      section.style.opacity = val;
+	      requestAnimationFrame(fade);
+	    }
+	  })();


La construcción de fade() es conocida como IIFE y sirve para llamar a una función inmediatamente luego de la definición.

http://benalman.com/news/2010/11/immediately-invoked-function-expression/
https://en.wikipedia.org/wiki/Immediately-invoked_function_expression

Dado que fade es llamada cuando se llama a fadeIn en la práctica es lo mismo usar una IIFE o poner el cuerpo de la misma dentro de fadeIn

sebadlf · 2017-04-27T18:20:55Z

01-JS-DOM-APIs/index.html

+	  })();
+	}
+
+	fadeIn(section);


Para fadeIn si podrías usar una construcción de IIFE ya que es la única llamada de la misma y sucede justo después de su definición

sebadlf · 2017-04-27T18:23:45Z

01-JS-DOM-APIs/index.html

+
+<!-- Function to set specific content to the section when is clicked. Exercise 6-->
+ <script>
+	function loadXMLDoc() {


No encuentro donde estas invocando esta funcion

Facundo97Maldonado · 2017-04-27T18:49:48Z

Respondi por skype pero no se si lo viste seba. Las funciones las llamo cuando hice el ejercicio. Despues, cuando pase al siguiente. No hice un boton por cada ejercicoo. A medida que iban funcionando le pasaba otro nombre de funcion. Lo del fadein lo tengo que cambiar? O es mas una mejor forma de hacerlo? El 27/04/2017 15:28, "Sebastián de la Fuente" <notifications@github.com> escribió:

…

***@***.**** commented on this pull request. ------------------------------ In 01-JS-DOM-APIs/css/estilosX.css <#2 (comment)> : > @@ -0,0 +1,28 @@ +.x1{ Estos estilos no los veo aplicados en ningún lado ------------------------------ In 01-JS-DOM-APIs/index.html <#2 (comment)> : > @@ -0,0 +1,211 @@ +<!DOCTYPE html> +<html> +<head> + <meta http-equiv="X-UA-Compatible" content="IE=edge"> + <meta http-equiv="Content-Type" content="text/html;charset=utf-8"> + <meta name="viewport" content="width=device-width, initial-scale=1"> + + <title>Topic 1</title> + + <link rel="stylesheet" type="text/css" href="myStylesheet.css"> +</head> +<body> +<script src="https://ajax.googleapis.com/ajax/libs/angularjs/1.4.8/angular.min.js"></script> Angular? estimo que esto esta de más ------------------------------ In 01-JS-DOM-APIs/index.html <#2 (comment)> : > + </button> + <input type="text" name="inputSearch" id="searchField"> + </div> + <br> + <div> + <button onclick="tableCreator()" id="tableMaker">Make a table</button> + <table id="table"> + + </table> + </div> + + + + +<script> + function changeText(id){ Es solo un tema de nombre id es en realidad el elemento del DOM que representa el botton. Probablemente "target" o "element" serian nombre mas adecuados ------------------------------ In 01-JS-DOM-APIs/index.html <#2 (comment)> : > +</script> + +<script> + let section = document.querySelector('.js-fade'); + + function fadeIn(section, display){ + section.style.opacity = -10; //time after page loads to display the content + section.style.display = display || "block"; + + (function fade() { + let val = parseFloat(section.style.opacity); + if (!((val += .1) > 1)) { + section.style.opacity = val; + requestAnimationFrame(fade); + } + })(); La construcción de fade() es conocida como IIFE y sirve para llamar a una función inmediatamente luego de la definición. http://benalman.com/news/2010/11/immediately-invoked-function-expression/ https://en.wikipedia.org/wiki/Immediately-invoked_function_expression Dado que fade es llamada cuando se llama a fadeIn en la práctica es lo mismo usar una IIFE o poner el cuerpo de la misma dentro de fadeIn ------------------------------ In 01-JS-DOM-APIs/index.html <#2 (comment)> : > + let section = document.querySelector('.js-fade'); + + function fadeIn(section, display){ + section.style.opacity = -10; //time after page loads to display the content + section.style.display = display || "block"; + + (function fade() { + let val = parseFloat(section.style.opacity); + if (!((val += .1) > 1)) { + section.style.opacity = val; + requestAnimationFrame(fade); + } + })(); + } + + fadeIn(section); Para *fadeIn* si podrías usar una construcción de IIFE ya que es la única llamada de la misma y sucede justo después de su definición ------------------------------ In 01-JS-DOM-APIs/index.html <#2 (comment)> : > + + (function fade() { + let val = parseFloat(section.style.opacity); + if (!((val += .1) > 1)) { + section.style.opacity = val; + requestAnimationFrame(fade); + } + })(); + } + + fadeIn(section); +</script> + + + <script> + function loadXMLDoc() { No encuentro donde estas invocando esta funcion — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#2 (review)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AabCd8-w1ojnKPDsD4DRB_zPIElK2PQ2ks5r0N5DgaJpZM4NJO8A> .

Facundo97Maldonado · 2017-04-27T18:50:38Z

Como serian nombres no horripilantes javi? jajaja El 27/04/2017 15:28, "Sebastián de la Fuente" <notifications@github.com> escribió:

…

***@***.**** commented on this pull request. ------------------------------ In 01-JS-DOM-APIs/css/estilosX.css <#2 (comment)> : > @@ -0,0 +1,28 @@ +.x1{ Estos estilos no los veo aplicados en ningún lado ------------------------------ In 01-JS-DOM-APIs/index.html <#2 (comment)> : > @@ -0,0 +1,211 @@ +<!DOCTYPE html> +<html> +<head> + <meta http-equiv="X-UA-Compatible" content="IE=edge"> + <meta http-equiv="Content-Type" content="text/html;charset=utf-8"> + <meta name="viewport" content="width=device-width, initial-scale=1"> + + <title>Topic 1</title> + + <link rel="stylesheet" type="text/css" href="myStylesheet.css"> +</head> +<body> +<script src="https://ajax.googleapis.com/ajax/libs/angularjs/1.4.8/angular.min.js"></script> Angular? estimo que esto esta de más ------------------------------ In 01-JS-DOM-APIs/index.html <#2 (comment)> : > + </button> + <input type="text" name="inputSearch" id="searchField"> + </div> + <br> + <div> + <button onclick="tableCreator()" id="tableMaker">Make a table</button> + <table id="table"> + + </table> + </div> + + + + +<script> + function changeText(id){ Es solo un tema de nombre id es en realidad el elemento del DOM que representa el botton. Probablemente "target" o "element" serian nombre mas adecuados ------------------------------ In 01-JS-DOM-APIs/index.html <#2 (comment)> : > +</script> + +<script> + let section = document.querySelector('.js-fade'); + + function fadeIn(section, display){ + section.style.opacity = -10; //time after page loads to display the content + section.style.display = display || "block"; + + (function fade() { + let val = parseFloat(section.style.opacity); + if (!((val += .1) > 1)) { + section.style.opacity = val; + requestAnimationFrame(fade); + } + })(); La construcción de fade() es conocida como IIFE y sirve para llamar a una función inmediatamente luego de la definición. http://benalman.com/news/2010/11/immediately-invoked-function-expression/ https://en.wikipedia.org/wiki/Immediately-invoked_function_expression Dado que fade es llamada cuando se llama a fadeIn en la práctica es lo mismo usar una IIFE o poner el cuerpo de la misma dentro de fadeIn ------------------------------ In 01-JS-DOM-APIs/index.html <#2 (comment)> : > + let section = document.querySelector('.js-fade'); + + function fadeIn(section, display){ + section.style.opacity = -10; //time after page loads to display the content + section.style.display = display || "block"; + + (function fade() { + let val = parseFloat(section.style.opacity); + if (!((val += .1) > 1)) { + section.style.opacity = val; + requestAnimationFrame(fade); + } + })(); + } + + fadeIn(section); Para *fadeIn* si podrías usar una construcción de IIFE ya que es la única llamada de la misma y sucede justo después de su definición ------------------------------ In 01-JS-DOM-APIs/index.html <#2 (comment)> : > + + (function fade() { + let val = parseFloat(section.style.opacity); + if (!((val += .1) > 1)) { + section.style.opacity = val; + requestAnimationFrame(fade); + } + })(); + } + + fadeIn(section); +</script> + + + <script> + function loadXMLDoc() { No encuentro donde estas invocando esta funcion — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#2 (review)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AabCd8-w1ojnKPDsD4DRB_zPIElK2PQ2ks5r0N5DgaJpZM4NJO8A> .

riledhel · 2017-04-28T20:54:35Z

01-JS-DOM-APIs/index.html

+		section.style.opacity = -10; //time after page loads to display the content
+ 	    section.style.display = display || "block";
+ 
+ 	  	(function fade() {


La idea acá es no hacer la animación a mano si no usar javascript para registrar un handler, eventos del browser para saber cuándo cargó y CSS3 para la transición

riledhel · 2017-04-28T21:00:06Z

01-JS-DOM-APIs/index.html

-    };
-    eventRequestReusable(config).then(resolve,reject); //Here is where the configuration object is entered
+	    function resolve(xhttp) {
+	   	   let response = JSON.parse(event.target.response);


de dónde sale event?? funciona, pero no veo por qué

Facundo97Maldonado added 10 commits April 19, 2017 16:04

Topic 1. From exercise 1 to 7/8

1353d7f

Revert "Topic 1. From exercise 1 to 7/8"

b9f1eb9

This reverts commit 1353d7f.

Topic 1. From exercise 1 to 7/8

09b928a

Topic 0. Point 1

ec856e2

Topic 0. Point 1.2

246ea0f

Topic 0. Done

6ffe43c

Topic 1 - From 1 to 8

1b942a1

Topic 1 - Exercise 9 Done

1fe0714

Topic 1 - Exercise 10. Done

f4e1511

Topic 1. Done

7cd96ec

sebadlf reviewed Apr 27, 2017

View reviewed changes

Buttons added, to call all the functions - Topic 1

c3906fa

riledhel reviewed Apr 28, 2017

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Topic1#2

Topic1#2
Facundo97Maldonado wants to merge 11 commits intomasterfrom
Topic1

Facundo97Maldonado commented Apr 26, 2017

Uh oh!

sebadlf Apr 27, 2017

Uh oh!

riledhel Apr 27, 2017

Uh oh!

sebadlf Apr 27, 2017

Uh oh!

sebadlf Apr 27, 2017

Uh oh!

sebadlf Apr 27, 2017

Uh oh!

sebadlf Apr 27, 2017

Uh oh!

sebadlf Apr 27, 2017

Uh oh!

Facundo97Maldonado commented Apr 27, 2017 via email

Uh oh!

Facundo97Maldonado commented Apr 27, 2017 via email

Uh oh!

riledhel Apr 28, 2017

Uh oh!

riledhel Apr 28, 2017

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Conversation

Facundo97Maldonado commented Apr 26, 2017

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Facundo97Maldonado commented Apr 27, 2017 via email

Uh oh!

Facundo97Maldonado commented Apr 27, 2017 via email

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants