Para empezar, felicidades a todos los que participaron en la competencia. Se nota el esfuerzo que hay detrás de los trabajos.
Aunque no sé mucho de esto quisiera aportar un poco a la competencia comentando principalmente sobre los códigos utilizados (que ya saben que el diseño no es lo mío
). Si notan algo raro o incorrecto, no duden en mencionarlo.
-----------------------------------------------------------------
Primer Trabajo:
Triby Código XHTML:
Muy legible. Bien indentado, usando el tabulador.
Hay algunos problemas con la semántica del código. El principal problema es el uso de <div>. Hay que tener en cuenta que ese elemento casi no tiene significado semántico, no es más que un contenedor o división de la página. Por ejemplo, el calendario, está hecho completamente con divs. Según mi interpretación, en este caso sería válido usar tablas, ya que el calendario básicamente es contenido tabulado. Así también, en el index, los textos debajo de los encabezados, son párrafos (<p>), no deberían ser <div>; lo mismo para el texto de los eventos que explica lo que se realiza en cada fecha (en la barra lateral derecha). También, existe ese problema en el formulario, donde la mayoría de los campos están envueltos con un div, podrías haber aplicado los estilos directamente al input.
Hay también un pequeño problema de inconsistencia en el uso de los encabezados (hX). Por ejemplo, el encabezado "Otros Diseños" de la barra lateral izquierda es un h4, lo que me haría pensar que tiene relación con el contenido principal, y que es menor en jerarquía; pero no hay mucha relación, por lo que probablemente sería más correcto un h2. Lo mismo con la barra lateral derecha del index, que es un h4, cuando no tiene mucha relación de jerarquía con los subtítulos "Diseño", "Manifiesto", etc. Podría ser un h3 considerando la relación de jerarquía con el subtítulo de la sección.
Sobre el código CSS no tengo mucho que comentar. Me gustó que comentaste bastante el código, lo que facilita la lectura, saber qué hace cada cosa. Me pareció curioso ver que usaste una unidad de medida (px) en los ceros
.
Segundo trabajo:
XLogus
<offtopic type="hablando del diseño">Me gustó la selección de colores, y el detalle de los degradados, en cuestión de diseño es uno de los trabajos que más me ha gustado</offtopic>
En cuanto al código. Uso correcto de la anidación de listas para el menú.
Hay también alguno que otro div que según mi interpretación podrían usarse otras etiquetas que tengan mayor significación semántica, por ejemplo, el adorno de la paleta podría ser un <img>. Algo que me llamó mucho la atención fue que aunque en el index usaste correctamente los párrafos para el texto de Bienvenida (<p>), no fue así en la sección "Manifiesto" y el pie de página, donde el texto está dentro del div y los párrafos están separados por saltos de línea "forzados" (<br />). En el menú lateral de "Lo Nuevo" podrías usar una lista desordenada para los enlaces y usar un texto alternativo más descriptivo, ya que no hay ningún otro texto explicando las imágenes, así que sin ellas, teniendo como texto "pic1", "pic2", "pic3" el lector se perdería un poco (nota: quedó muy bien ese efecto de zoom al pasar el mouse).
En el formulario me pareció curioso el hecho de que el formulario está dentro del fieldset en lugar de que el fieldset esté dentro del formulario
. Aunque usaste correctamente las etiquetas <label> que sirven para especificar la función de cada campo del formulario, no especificaste el atributo for para que se sepa a qué campo está relacionada cada label (no haría falta si el campo estuviera dentro de la etiqueta label).
P.S.: No hay h1
Código CSS:
Tampoco tengo mucho para comentar. Se nota bien limpio. Aunque no hay muchos comentarios, se puede leer bien. Podrías haber usado la forma abreviada para padding y background en algunos casos (en lugar de padding-top, padding-right, padding-bottom y padding-left para el primer caso, y para el segundo en lugar de background-image, background-repeat y background-position).
Tercer trabajo:
jeybi
<offtopic type="hablando del diseño">Me gustó mucho el logo. Imagino el trabajo que tuviste para hacerlo
:
Nota: el texto del pie "Diseñador por..." casi no se nota ya que se mezcla con el color de la imagen de fondo.</offtopic>
Código XHTML:
Me gustó el código en general. Usaste correctamente los encabezados (hX). También de los párrafos para el texto y las listas desordenadas. Me llamó la atención esos <br /> en el formulario
, así como el fieldset sin legend.
Código CSS:
También me gustó el hecho de que comentaste varias secciones del código para poder ubicarse dentro del archivo.
Los selectores #menu span, #logo span y #centro h2 span hacen lo mismo. Podrías unirlos y comentarlos explicando que reemplazan el texto por la imagen de fondo.
El selector #contactoCont legend sobra, así como el #contactoCont legend span (el primero no se aplica a ningún elemento y el segundo está vacío).
Para el efecto de hacer mayor la primera letra en los párrafos podrías haber usado el pseudo-elemento first-letter en lugar del span.
-----------------------------------------------------------------------------------
Los otros trabajos voy a revisarlos después y dar mis comentarios.
Nota: Me quedé con las ganas de ver en alguno de los trabajos el uso del tag <acronym> (acrónimo) al envolver "XHTML" y "CSS" dentro del texto.
Saludos.