Skip to content

cipher#35

Open
adriana121 wants to merge 65 commits intoLaboratoria:masterfrom
adriana121:master
Open

cipher#35
adriana121 wants to merge 65 commits intoLaboratoria:masterfrom
adriana121:master

Conversation

@adriana121
Copy link
Copy Markdown

No description provided.

src/style.css Outdated
h2 { text-align: center; }
h3{ text-align: center; }
a { text-align: center; }
input {position: absolute;
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cuidemos la indentación y si repites en un mismo estilo para varios elementos, siempre es mejor una clase, recuerda, principio DRY

src/style2.css Outdated
#name { background-color:#D17595; border-bottom:#F5E9F7; border-radius:20px; width: 100px; height: 100; align-content: center;}
#form {
text-align: center;
}
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Indentación

src/codigo.html Outdated
<input type="text" name="nombre" id="codigo" placeholder="Nombre"><br>



Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tu código no es muy legible, cuida la indentación y consistencia

src/cipher.js Outdated
}
}

function decifrar(){
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Procura tener consistencia en tu código, si empleas palabras en inglés continua con ellas, tu indentación debe mejorar, por todo lo demás llevas un excelente trabajo!

} No newline at end of file
body {
background-image: url("../src/helado\ fondo.jpg");
}
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bien!

src/index.html Outdated
<h4></h4>
</div>
<script src="index.js" type="module"></script>
</body>
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Adri! tu avance es muy bueno y me parece perfecto que ya tengas dominado el tema de git, lo que si es que procura que cada cambio que subas sea significativo, es decir, que no sea solo agregar una llave, una imagen o algo asi, que tenga mas sentido. Buen trabajo!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants