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

[MAD DE Virginia Majuelos] #74

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

VirginiaMajuelos
Copy link

"Ejercicio revera"

Este ejercicio me costó mucho al principio porque no entendia/contextualizaba boostrap, ni con ello qué hacer. A última hora de la noche me iluminé y empecé a entender "algo" y a partir de ahi lo poquito que me dió tiempo para poder descansar algo.

Un saludo

@ta-web-mad
Copy link
Owner

Hola Virginia,

Lo que dices es normal, muchas veces al principio no entiendes nada y cuesta un rato hasta que de repente se te enciende la bombilla. A veces es un poco frustrante, pero cuando al fin lo entiendes es genial! Al final es siempre lo mismo, entender que dentro de row va col, y esa parte parece que la terminaste por entender, eso eso bueno.

Te comento un par de cosillas:

  • Cuidado con la indentación, como dijimos ayer en clase, tener un código limpio y ordenado es importante tanto para uno mismo como para el resto, facilita la lectura y, cuando algo falla, ayuda mucho.

Por ejemplo, así está esto ahora (sobran muchas tabulaciones dobles por ahí):
image

Así sería más correcto:
image

Los span no se te ven en rojo porque en el css has puesto .span, sobra el punto, ya que no es una clase! si lo quitas verás que funciona bien.

Un truquillo para que las imágenes no se deformen (por ejemplo la del gatito y el perrete de abajo), prueba a añadir esto en tu código y ver lo que pasa, esta propiedad os servirá en muchas ocasiones para evitar eso:
image

Seguimos! 🔥

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.

2 participants