Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Designed qrcode challenge using html and css.

Rakshith Jโ€ข 240

@rakshithjodukallu

Desktop design screenshot for the QR code component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


This is the simple qrcode task that completed in 30mins. add the text column wise and make the div block center. Give me Your valuable comments, Thank you

Community feedback

Lucas ๐Ÿ‘พโ€ข 104,420

@correlucas

Posted

๐Ÿ‘พHi @rakshithjodukallu, congratulations on your first solution!๐Ÿ‘‹ Welcome to the Frontend Mentor Coding Community!

Great solution and a great start! From what I saw youโ€™re on the right track. Iโ€™ve few suggestions for you that you can consider adding to your code:

Remove the margin-top and align the whole content using just the body.See the code below:

body {
    min-height: 100vh;
    background-color: #d5e1ef;
    margin-top: 200px;
    display: flex;
    align-items: center;
    justify-content: center;
    flex-direction: column;
}

โœŒ๏ธ I hope this helps you and happy coding!

Marked as helpful

0

Rakshith Jโ€ข 240

@rakshithjodukallu

Posted

@correlucas Your code working, thanks for suggestion.

0
Nelson Nzewiโ€ข 270

@nzewi

Posted

Great solution there Rakshith J

1.<img src="./images/image-qr-code.png" class="img-fluid" alt=""> You can add a value to the alt tag because the image is not merely decorative but conveys something meaningful.

2.<div class="codeouter"> It would be good if you used more descriptive class names to help other developers understand your code better and also help you in the future to understand.

3.Use <main> instead of a simple <div> . This way, you improve the semantics and accessibility of your site, showing which is the main block of content on this page. Every page should have a <main> block

4.The main heading has the tag <h4>. You should replace it with <h1> since this heading is the main title on this page. Every page should have one <h1> to declare the most important title and you should follow the hierarchy using the heading sequence (h1, h2, h3, h4, h5) and never jump a level.

Marked as helpful

0

Lucas ๐Ÿ‘พโ€ข 104,420

@correlucas

Posted

@nzewi Amazing and original review.

0
Rakshith Jโ€ข 240

@rakshithjodukallu

Posted

@nzewi I will make add these tips further design. thanks mate

0

Please log in to post a comment

Log in with GitHub
Discord logo

Join 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