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

qr-code-component

Rakesh Kumarβ€’ 20

@Rakesh709

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


Can you give me the feebback regarding my css which i use.

Community feedback

Naveen Gumasteβ€’ 10,460

@NaveenGumaste

Posted

Hay ! Rakesh Kumar Good Job on challenge

These below mentioned tricks will help you remove any Accessibility Issues

-> Add Main tag after body <main class="container"></main>

-> Recheck the background

-> Learn more on accessibility issues

If this comment helps you then pls mark it as helpful!

Have a good day and keep coding πŸ‘!

Marked as helpful

0
Dinesh Sakeβ€’ 320

@Nrupatungan

Posted

Hello Rakesh, congrats on completing your challenge.πŸŽ‰

I saw your solution and i recommend you to follow these steps to better your Frontend:

  1. Use Semantic HTML as it gives the content on the page meaning and structure by using the proper element. As regular <div> and <span> elements don’t hold any semantic value. They exist for styling purposes only. check this link for more details click here.
  2. Properly center your container or main-component by using methods like positioning, Flexbox align-items etc. You can search on google on how to center containers.
  3. Use Google fonts for styling your fonts.
  4. Use correct Colors and Typography provided in your style-guide as it will give a great look to your webpage.

It would help you a lot if you adopt these tips in your upcoming Frontend Challenges.

Have a great day!!

Marked as helpful

0
Travolgi πŸ•β€’ 31,420

@denielden

Posted

Hi Rakesh, congratulations on completing the challenge.

I had a look at your solution:

  • add main tag and wrap the card for Accessibility
  • try to remove all margin from container and middle-container classes and use flexbox to the body for center the card. Read here -> flex guide
  • also set heigth of body to 100vh because Flexbox aligns to the size of the parent container.

In the end do not use the css style inline but one of the classes :)

Overall you did well! Hope this help ;)

Marked as helpful

0

Account Deleted

Hello there! πŸ‘‹

Congratulations on finishing your challenge! πŸŽ‰

I have some feedback on this solution:

  • Always Use Semantic HTML instead of div like <main> <header> , etc for more info.

if my solution has helped you do not forget to mark this as helpful!

Marked as helpful

0

Rakesh Kumarβ€’ 20

@Rakesh709

Posted

@Old1337 yes yes thank you

1
Hexerβ€’ 3,660

@Phalcin

Posted

Great work man, check mine also.

0

Account Deleted

Hi there πŸ‘‹

Congratulate on finishing your project πŸŽ‰. You did a great job πŸ’‘

I give some suggestions to help you take your project design to the next level πŸ“ˆπŸ˜‰

  • Change the background color to #D5E1EF πŸ™Œ
  • They don't family is not imported too, change it to the one as given in the style-guide.md file
  • I'd suggest decreasing the padding in the card also πŸ‘

Happy coding β˜•

Maqsud

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