Skip to content
This repository was archived by the owner on Jul 11, 2026. It is now read-only.

MI PRIMER CODEREVIEW LAURA ALFONSO#25

Open
lauraac wants to merge 2 commits into
Laboratoria:masterfrom
lauraac:master
Open

MI PRIMER CODEREVIEW LAURA ALFONSO#25
lauraac wants to merge 2 commits into
Laboratoria:masterfrom
lauraac:master

Conversation

@lauraac

@lauraac lauraac commented Feb 11, 2020

Copy link
Copy Markdown

Hola estoy empezando con javascript

@HectorBlisS

Copy link
Copy Markdown

Es importante que subas el resto de tu trabajo para que los próximos code reviews añadan más valor a tu aprendizaje ya que no hay mucho que revisar ;) sigue trabajando!

@HectorBlisS HectorBlisS left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Es importante que subas el resto de tu trabajo para que los próximos code reviews añadan más valor a tu aprendizaje ya que no hay mucho que revisar ;) sigue trabajando!

Comment thread src/style.css
@@ -0,0 +1,70 @@
form {

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Todo esta en orden, pero podrias mejorar tus selectores y evtar tantas clases si usaras más "especificidad" (investiga un poco) para que uses padres y accedas a sus hijos con especificidad:

.container > div h2{
// styles
}

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants