Final#15
Conversation
rglepe
left a comment
rglepe
left a comment
There was a problem hiding this comment.
Falta el enlace a la raíz del documento dentro del título de la página.
Los div que usas para estructurar la página no son correctos. El menú de navegación no debería posicionarse junto a la imagen. La imagen forma parte del artículo y debería moverse junto a los párrafos dentro de un elemento semántico de tipo main o article.
El elemento main, entonces, debería posicionarse con respecto al elemento aside.
El elemento figcaption no necesita un elemento interno de título (h3) ya es un elemento preparado para llevar texto.
el elemento de lista ordenada ol no es necesario dentro de este ejemplo. Sobretodo si no lo utilizas para dar formato.
Recuerda incluir los mínimos elementos necesarios. Mejor estructurar con semánticos y una vez que estés con el layout de la página en el CSS puedes recurrir a elementos contenedores para posicionar, si fuera necesario.
la propiedad repeat debería tener el valor repeat-x;
el cambio de fondo sobre la barra de menu debe hacerse sobre el elemento no sobre el texto.
el layout no consigue igualar al ejemplo al no estar bien estructurado el html.
Creo que necesitas practicar bastante la estructuración de las páginas hasta que entiendas como utilizar los elementos semánticos y los contenedores para realizar el layout de la página.
|
Hola Raúl.
Avui i demà no podré venir.
Fins dilluns,
Jordi
De: rglepe [mailto:notifications@github.com]
Enviado el: miércoles, 1 de mayo de 2019 22:22
Para: bcncodes-training/tests <tests@noreply.github.com>
CC: idroj1966 <idroj66@hotmail.com>; Author <author@noreply.github.com>
Asunto: Re: [bcncodes-training/tests] Final (#15)
@rglepe commented on this pull request.
Falta el enlace a la raíz del documento dentro del título de la página.
Los div que usas para estructurar la página no son correctos. El menú de navegación no debería posicionarse junto a la imagen. La imagen forma parte del artículo y debería moverse junto a los párrafos dentro de un elemento semántico de tipo main o article.
El elemento main, entonces, debería posicionarse con respecto al elemento aside.
El elemento figcaption no necesita un elemento interno de título (h3) ya es un elemento preparado para llevar texto.
el elemento de lista ordenada ol no es necesario dentro de este ejemplo. Sobretodo si no lo utilizas para dar formato.
Recuerda incluir los mínimos elementos necesarios. Mejor estructurar con semánticos y una vez que estés con el layout de la página en el CSS puedes recurrir a elementos contenedores para posicionar, si fuera necesario.
la propiedad repeat debería tener el valor repeat-x;
el cambio de fondo sobre la barra de menu debe hacerse sobre el elemento no sobre el texto.
el layout no consigue igualar al ejemplo al no estar bien estructurado el html.
Creo que necesitas practicar bastante la estructuración de las páginas hasta que entiendas como utilizar los elementos semánticos y los contenedores para realizar el layout de la página.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub<#15 (review)>, or mute the thread<https://github.com/notifications/unsubscribe-auth/AL23F2LVXYWK2D3O4ICU7YDPTH3X3ANCNFSM4HJJFIWQ>.
|
2 participants
No description provided.