Skip to content

Robby test#10

Open
naranjito72 wants to merge 8 commits into
cifo-training:masterfrom
naranjito72:master
Open

Robby test#10
naranjito72 wants to merge 8 commits into
cifo-training:masterfrom
naranjito72:master

Conversation

@naranjito72

Copy link
Copy Markdown

No description provided.

@rglepe rglepe left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Buen ejercicio. Te hago algunos comentarios:

Falta el contenedor para fijar las propiedades del fondo blanco y el ancho 80%.

El enlace a la raíz del doc es "./"
Diría algo como p-itis. No usas correctamente el elemento p.
Te copio la definición del elemento de la doc oficial: líneas y párrafos:
Tradicionalmente los autores dividen sus pensamientos y argumentos en secuencias de párrafos. La organización de la información en párrafos no se ve afectada por el modo en que se presentan los párrafos: los párrafos que están doblemente justificados contienen los mismos pensamientos que aquellos que están justificados a la izquierda. La forma en que HTML define un párrafo es directa: el elemento P define un párrafo.

falta el elemento nav

repites la propiedad text-align en h1 y a
position:relative sin coordenadas es lo mismo que no posicionar (caso de menu_nav)

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.

3 participants