Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merge branch 'main' of https://github.com/davferod/hacktoberfest-2022 #295

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

davferod
Copy link

@davferod davferod commented Oct 8, 2022

⭐ Descripción

Por favor, incluye una breve descripción de tus cambios.
modulos completamente aislados html, css y js

🚀 Tipo de cambio

  • [x ] Nueva característica (cambios que no rompen nada y agregan funcionalidad).

🤔 ¿Cómo ha sido probado este cambio?

un proyecto desde cero con html y js muy basico

Describa las pruebas que ha realizado para verificar los cambios. Proporcione
instrucciones para que podamos reproducirlos. Por favor, indique también cualquier detalle relevante
para la configuración de su prueba. Ejemplo:

  • Prueba A
  • Prueba B

Configuración de pruebas:

  • Versión de Node: 16.13
  • Versión de Astro: 1

📝 Lista de verificación:

(*) -> obligatorio.

  • Mi código sigue las reglas de este proyecto *.
  • He creado mis propias clases / ids y he comprobado que no afectan a otros diseños de la comunidad *.
  • He realizado una auto-revisión de mi propio código *.
  • [x ] He comentado mi código, particularmente en las áreas difíciles de entender.
  • Mis cambios no generan nuevos avisos *.
  • No he instalado nuevos módulos con npm install o yarn install *.
  • La solución es de mi autoría y no he copiado código de otros participantes *.
  • No modifique ningun otro archivo más que los míos *.

@vercel
Copy link

vercel bot commented Oct 8, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
hacktoberfest-2022 ✅ Ready (Inspect) Visit Preview Oct 9, 2022 at 8:41PM (UTC)

@davferod
Copy link
Author

davferod commented Oct 8, 2022

un codigo muy basico en html.css y javascript

@@ -0,0 +1,227 @@
* {
Copy link
Owner

Choose a reason for hiding this comment

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

Los estilos deben tener un scope para que sólo se apliquen a tu solución.

Puedes añadir la id #davferod y hacer esto:

#davferod * {}

Si no, entrará en conflicto con las otras soluciones.

Copy link
Owner

@midudev midudev left a comment

Choose a reason for hiding this comment

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

Los estilos necesitan un scope.

@davferod
Copy link
Author

davferod commented Oct 9, 2022

ya realicé las modificaciones estuve validando que no afectara a los demás, disculpa soy demasiado novato en esto, pero enserio tengo muchas ganas de participar de estas cosas para aprender afrontar estos desafíos

@pheralb
Copy link
Collaborator

pheralb commented Oct 10, 2022

🚧 .container es una clase que se repite bastante en otros proyectos de la comunidad: https://github.com/midudev/password-generator/pull/295/files#diff-f4793a659b1ff12d9835d2c22bba34a81e7e2cdc5ef0134782b4b6a8a11620e1R18. Por favor, crea clases personalizadas con tu nombre de usuario de github para evitar conflictos:

 .davferod-container {
	max-width: 500px;
	width:90%;
	margin: 50px auto;
}

También añade esa modificación a clases tipo .btn, .app... => https://github.com/midudev/password-generator/pull/295/files#diff-f4793a659b1ff12d9835d2c22bba34a81e7e2cdc5ef0134782b4b6a8a11620e1R61 🤔

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.

4 participants