Revisión Dianyela Maldonado#38
Open
DianyelaMaldonado wants to merge 44 commits intoLaboratoria:masterfrom
Open
Revisión Dianyela Maldonado#38DianyelaMaldonado wants to merge 44 commits intoLaboratoria:masterfrom
DianyelaMaldonado wants to merge 44 commits intoLaboratoria:masterfrom
Conversation
… contiene, el formulario pero sin funcionalidad,
…on estilos de prueba,no son los finales
…, le di funcionalidad a los botones en index.js, agregué pocos cambios al Readme.md aún me falta acabar el readme.md
…taba el proyecto como tal, y agregué fotos de los prototipos en papel, como el final
…do en el commit anterior a este pero no, ya le realice los cambios necesarios
…ón a los botones de index.js para que funcionen en conjunto, solo me funciona en la consola, por ahora lo guardo así mientras intento hacer que ya funcione en el contenido visible de la página, no logro esa interacción todavía
… cipher.js en el archivo ticketspersonalizados.html guardé div que me da el resultado al pulsar el (generar código)
…nto mi cifrado y descifrado se ven en la iteracción
…magenes nuevas en el slider,coloqué animacion de Latir al logo usando keyframes
AdrianaHY
reviewed
Feb 14, 2020
src/style.css
Outdated
| .botooon, | ||
| .botooonc, | ||
| .botooond { | ||
| background: #96A0AF; |
There was a problem hiding this comment.
No es necesario definir diferentes clases, una clase se puede compartir en diferentes elementos, es decir, puedes definir una clase button y compartirla con los elementos que se necesiten. Se mas clara con tus clases, botoooooon no es un buen nombre :)
AdrianaHY
reviewed
Feb 14, 2020
AdrianaHY
left a comment
AdrianaHY
left a comment
There was a problem hiding this comment.
Buen avance! Por favor, sigue así, cuida tus buenas prácticas :)
AdrianaHY
reviewed
Feb 15, 2020
src/index.html
Outdated
| <!-- MENÚ de navegación --> | ||
| <!-- la etiqueta NAV representa todos los links de navegación que vamos a utizar--> | ||
| <section id="MenuDeNavegación">s</section> | ||
| <section id="MenuDeNavegación"></section> |
There was a problem hiding this comment.
Los comentarios y los espacios son innecesarios.
AdrianaHY
reviewed
Feb 15, 2020
|
|
||
| /* color de fondo de la página #C4C4C4 */ | ||
|
|
||
| header { |
…ml para cards, de igual agregué en style.css cambios para las cards antes mencionadas
…lizados.html cambie texto dentro de h2
…ás, organizando de esta manera mis buenas practicas
…nt-changes Feature/ updating relevant changes
…nt-changes add changes in readme, new images
…nt-changes chenge images brokenin readme.md
…nt-changes change image cipher tablet
…nt-changes add changes in images tablet again
add new changes in Readme.md
add changes in url images of Readme.md
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
saludos :)