Open adawolfs opened 2 weeks ago
Ya lo revise todo, gracias por la información voy a poner en marcha los cambios y lo estaré subiendo.
Vale, con respecto al HTML
Considera utilizar más elementos HTML5 para mejorar la accesibilidad y la estructura del documento.
Se puede usar
Cambia
Evita el uso de separadores como elementos de lista, en el menú de navegación, estás usando
Un botón dentro de un enlace es redundante y podría ser simplificado solo a un enlace con estilos de botón.
https://github.com/GuateGeeks/page-stage/blob/develop/my-static-site/index.html#L8 No uses referencias absolutas, usa referencias relativas
https://github.com/GuateGeeks/page-stage/blob/develop/my-static-site/index.html#L30 Usar clases para crear este menu no es muy estandar,recomendaria que uses mejor el selector "nav div"
https://github.com/GuateGeeks/page-stage/blob/develop/my-static-site/index.html#L40 Veo que usas muchas clases especificas, podrias intentar hacer clases mas generales, podrias tener lo siguiente
<section id="contact">
<div class="container">
<div class="content">
Y en tu css hacer selecciones por el ID
....
Recomiendo que saques el archivo index de "my-static-file" y ponlo en la raiz del proyecto para simplificar la estrucutra.
Se usa font-family: 'Barlow'; en múltiples lugares. Podrías simplificar este proceso aplicando la fuente a nivel más global (por ejemplo, en body) y sobreescribir donde sea necesario.
Para colores, tamaños de fuente y otros valores que se repiten, considera utilizar variables CSS. Esto hace que las actualizaciones y mantenimiento sean más fáciles y rápidas. Por ejemplo: css Copiar código :root { --main-font: 'Barlow', sans-serif; --primary-color: #B230C8; --secondary-color: #172B60; --text-color: #fff; --font-size-large: 40px; --font-size-normal: 20px; } body { font-family: var(--main-font); color: var(--text-color); }
Elimina cualquier redundancia en los selectores y propiedades. Por ejemplo, si múltiples secciones comparten estilos comunes, crea clases reutilizables. Reduce la especificidad donde sea posible para hacer el CSS más fácil de sobreescribir y mantener.
Aumentar la Semántica en CSS:
En lugar de nombres de clase como .box o .big-box, considera nombres más descriptivos que reflejen su función o contenido, lo cual mejora la legibilidad y mantenimiento del código.
Considera usar propiedades como grid o flexbox para diseños más complejos, lo cual puede simplificar tu código y mejorar la responsabilidad y mantenibilidad del layout.
Estructura de Carpetas
Seguir una estructura convencional para las carpetas
Nomenclatura
Nombres de Archivos y Carpetas:
-
para separar palabras.style.css
,main.js
,logo.png
.CSS:
style.css
,reset.css
,layout.css
.JavaScript:
main.js
,utils.js
.Imágenes:
logo.png
,background.jpg
.Fuentes:
myfont.woff
,myfont.woff2
.Uso de Referencias Relativas sobre Absolutas
Referencias Relativas:
Referencias Absolutas:
Estándares para Commits
Para mantener un historial de cambios claro y comprensible, es importante seguir una convención para los mensajes de commits:
Formato del Mensaje:
Tipos de Commits:
feat
: Una nueva característica.fix
: Arreglo de un bug.docs
: Cambios en la documentación.style
: Cambios que no afectan el código (formateo, espacios en blanco, etc.).refactor
: Cambios en el código que no arreglan un bug ni añaden una característica.test
: Añadir o corregir pruebas.chore
: Actualización de tareas de build, configuración, etc.Ejemplos de Mensajes de Commits:
feat: add responsive navigation bar
fix: correct typo in index.html
docs: update README with project setup instructions
style: reformat CSS files
refactor: reorganize JavaScript functions
test: add unit tests for main.js
chore: update dependencies
Buenas Prácticas Adicionales
Comentarios en el Código:
Consistencia:
Documentación:
README.md
en la raíz del proyecto explicando la estructura, cómo configurar y ejecutar el proyecto, y cualquier otra información relevante.Usar referencias relativas sobre absolutas