Melvin Aguilar π§π»βπ»β’ 61,220
@MelvinAguilar
Posted
Hello there π. Good job on completing the challenge !
I have some suggestions about your code that might interest you.
- Instead of using pixels in font-size, use relative units like
em
orrem
. The font-size in absolute units like pixels does not scale with the user's browser settings. Resource π.
- Use
min-height: 100vh
instead ofheight
. Setting the height to 100vh may result in the component being cut off on smaller screens, such as a mobile phone in landscape orientation.
- Maintain a clear heading hierarchy, starting with <h1> and progressing through heading levels. Source π
- Add an
alt
attribute to the QR code image, explaining its purpose, e.g.,QR code to frontendmentor.io
.
I hope you find it useful! π Above all, the solution you submitted is great!
Happy coding!
Marked as helpful
1
P
Scott Rogenerβ’ 50
@scottyroges
Posted
@MelvinAguilar Thank you for the tips!
0