@MelvinAguilar
Posted
Hello there ๐. Good job on completing the challenge !
I have some suggestions about your code that might interest you.
- The
alt
attribute should not contain the words "image", "photo", or "picture", because the image tag already conveys that information.
- Instead of
h-screen
(height: 100vh), usemin-h-screen
(min-height: 100vh) as the class for the body element.min-h-screen
ensures that the body element takes at least the full height of the viewport, which is useful for content that might exceed the screen height.
I hope you find it useful! ๐ Above all, the solution you submitted is great!
Happy coding!
Marked as helpful
@yogskr
Posted
@MelvinAguilar
Hey, thanks for the feedback and suggestions!
- You're right about the
alt
attribute. I'll change it for usre. - I totally didn't think about using
min-h-screen
at all. Thanks for pointing that out.
Cheers!