@vanzasetia
Posted
Hello, @Meteo-Flagyl00! 👋
Congratulations on completing your first Frontend Mentor challenge! 🎉
There are a lot of things that you can improve.
- First, I recommend using flexbox to position to the middle of the page.
position: fixed
makes the card out of the normal flow. As a result, it makes it hard to style. - The only thing that the card needs to be responsive is a
max-width
. You only need to prevent the card from becoming too large on a wide screen. For theheight
of it, let the content inside it dictate it. - If all the values of the
padding
are the same (padding: 10px 10px 10px 10px;
), you can make it shorter by writing it like this:padding: 10px;
. - I recommend improving the alternative text of the QR code to "QR code for frontendmentor.io". It's going to give the users clear information about the QR code.
Overall, great work. You manage to keep the specificity of the CSS as low as possible. So, keep learning and happy coding! 😄
Marked as helpful
@Meteo-Flagyl00
Posted
Hello, @vanzasetia Thank you so much for your time brother, I knew I had a lot of things that I need to work on I just didn't know what are they. I will try to work on the things u pointed out in the next challenges , and once again thank u so much for your time.
@vanzasetia
Posted
@Meteo-Flagyl00 No problem! 👍