Design comparison
Solution retrospective
- What did you find difficult while building the project?
- Which areas of your code are you unsure of?
- Do you have any questions about best practices?
Community feedback
- @MelvinAguilarPosted over 1 year ago
Hello π. Congratulation on successfully completing your first challenge π ! !
I have other recommendations regarding your code that I believe will be of great interest to you.
HTML π·οΈ:
- Always avoid skipping heading levels; Starting with <h1> and working your way down the heading levels (<h2>, <h3>, etc.) helps ensure that your document has a clear and consistent hierarchy. Source π
- 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 π¨:
- 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.
- Remove the width of the
.qr-continer
element, let it occupy 100% of the screen width. When you set a fixed width, it will only work well when the viewport (1440 and 375px) has the same width, this means that the website will not adapt to the different screen sizes, and will look bad when it is accessed on a smaller device.
I hope you find it useful! π Above all, the solution you submitted is great!
Happy coding!
Marked as helpful2 - @frank-itachiPosted over 1 year ago
Hello there π. You did a good job!
I have some suggestions about your code that might interest you.
HTML π:
- Wrap the page's whole main content in the
<main>
tag. - If your code has different sections that have a specific like a , , sections or footer, itβs a good practice to enclose those parts with HTML5 landmarks.
For example, you could use a
<footer>
tag to wrap a footer section. - The heading order is important in the html structure so try to always start your headings and/or titles with an
<h1>
tag and then you can increase by one if you need to use more headings in your html code.
I hope you find it useful! ππ Above all, the solution you submitted is greatπ!
Happy
<coding />
π!Marked as helpful2 - Wrap the page's whole main content in the
- @beRajeevKumarPosted over 1 year ago
Congratulations Bro, you complete your first challenge.My advise to your is that You must think about solve the project and learn as much as possible form the project not think about just finish the project.
1
Please log in to post a comment
Log in with GitHubJoin our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord