Está listo mi análisis. Lo incluyo en 2 mensajes porque el límite de imágenes x mensaje es 15 y uso muchos emoticons
Felicidades a ambos por participar, pues independientemente del resultado participar y aceptar la discusión pública de sus trabajos ya implica un gran esfuerzo.
-------------------------------------------------------------------------- Buzu CONSIDERACIONES GENERALES
-
El CSS y HTML validan y además la codificación usada es utf-8
-
No tiene las 2 páginas internas requeridas en las reglas (tiene solo 1 pero que tampoco se diferencia demasiado de la portada del sitio) y además tienes MUCHAS faltas ortográficas.
-
Hecho de menos el uso de algún otro elemento en el contenido del sitio, solo hay un párrafo, una imagen y un formulario, nada más.
HTML
-
Buen indentado y comentado
-
El menú es una lista desordenada de enlaces, bien.
-
Todos los títulos están al mismo nivel (H1) cuando deberían estar con una relación título principal-subtítulo (el título de la sección "Nosotros" por ejemplo está claro que debería de ser un subtítulo del título de la página, y no estar al mismo nivel). Además aunque no es obligatorio siempre se sugiere que el título de la página sea un enlace a la portada del sitio.
-
Los imputs del formulario no tienen labels
-
Error grave: muchas imágenes decorativas están puestas en el html y no en el css (flechas del menú y la cabecera). Si miras la página sin css las imágenes aparecen, aunque no dan ninguna información, para el título de la página ya está el H1 (por lo tanto el título aparece repetido, una vez el H1 y otra vez en la imagen) y la flecha del menú al estar repetida en cada li, aparece 8 veces, por lo que tampoco da información de cual es la página seleccionada. Si lees el menú con un lector de voz, lo que te va a leer es "Home Imagen de una flecha sin importancia. Nosotros Imagen de una flecha sin importancia. La Experiencia Imagen de una flecha sin importancia...." lo que lo hace súper tedioso.
CSS
-
aunque abreviaste propiedades como margin o padding, no hiciste lo mismo con las propiedades de background. Otra cosa que te faltó es definir un tipo de fuente por defecto.
-
el uso de posición absoluta para posicionar las imágenes. Sin considerar el error de incluir las img en el html, para que la cabecera te quedara bien tan solo tenias que darle display:none al h1 del título. Vi que dices algo de eso en los comentarios, pero generalmente lo que se hace es en el H1 A darle por un lado display:block y poner la imagen como fondo, y por el otro para ocultar el texto lo puedes hacer con text-indent:-9999px. Si no te carga la imagen de fondo es porque no se está leyendo el CSS, y por consecuencia tampoco se leerá el text-indent, así que no corres riesgo de quedarte sin título
-
el enfoque del menú fue equivocado. A los enlaces le tendrías que haber dado el fondo gris junto con display:block (así todo el fondo gris sería un enlace, no solo el texto) y la flecha del item activo la tendrías que haber puesto como fondo del LI.activo. Además debido a que le diste un ancho fijo a los li, el texto de "Patrocinadores" al ser más grande se queda en parte fuera del li.
-
el uso de selectores tipo input[name="name"] y *... fue una grata sorpresa, aunque considero que el * quizás lo usaste de una forma algo arriesgada, ya que no solo afecta a los elementos hijos, sino que a los nietos, etc... en fin, a TODOS los elementos, los que están y los que podrían estar en un sitio funcional.