@MelvinAguilar
Posted
Hello there ๐. Good job on completing the challenge !
I have some suggestions about your code that might interest you.
HTML ๐ท๏ธ:
- Wrap the page's whole main content in the
<main>
tag.
- Since this component involves scanning the QR code, the image is not a decoration, so it must have an
alt
attribute. Thealt
attribute should explain its purpose. e.g.QR code to frontendmentor.io
CSS ๐จ:
- You can center the component without using
grid-template-columns
, all you need to add isplace-content: center
and it will be centered. In this solution you don't have any other elements so it's not very suitable to use columns.
- Use
min-height: 100vh
instead ofheight
. Setting the height to 100vh may result in the component being cut off on smaller screens if you useplace-content: center
, such as a mobile phone in landscape orientation.
I hope you find it useful! ๐ Above all, the solution you submitted is great!
Happy coding!
Marked as helpful
@aljager1983
Posted
@MelvinAguilar thanks for the tip, tried out your suggestions its simpler than my previous one, and less lines of codes, thanks for the tips