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

Responsive QR code

@SharonNabiryo

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 anyone please review my code and give me feedback on how I can optmize my solution.

Community feedback

Panjiβ€’ 2,110

@pperdana

Posted

Hello thereπŸ‘‹!! Congratulations on completing this challange.

  • I have some additional recommendations for your code that I think you'll find interesting and valuable.

πŸ“Œ Image element do not have alt attributes or you leave it blank

for example code

<img src="images/image-qr-code.png">

In this code you should add alt in your code

<img 
  src="images/image-qr-code.png"   
  alt="qr code"
>
  • This alt attribute provides alternative text for images, which is important for accessibility purposes. Screen readers, use the alt attribute to read out loud what the image is about, allowing visually impaired users to understand the content of the page.

I hope you found this helpful! 😊

Happy codingπŸ€–

Marked as helpful

0

@SharonNabiryo

Posted

Thank you so much @Panji200

0
Ecem Gokdoganβ€’ 9,380

@ecemgo

Posted

Some recommendations regarding your code could be of interest to you.

In order to fix the accessibility issue:

  • You need to replace <div class="container"> with the <main class="container"> tag and <div class="attribution"> with the <footer class="attribution"> tag. You'd better use Semantic HTML, and you can also reach more information about it from Using Semantic HTML Tags Correctly.

After committing the changes on GitHub and you need to deploy it as a live site. Finally, you should click generate a new report on this solution page to clear the warnings.

Hope I am helpful. :)

Marked as helpful

0

@SharonNabiryo

Posted

I appreciate the feedback. Thank you so much @ecemgo

1
Ecem Gokdoganβ€’ 9,380

@ecemgo

Posted

@SharonNabiryo anytime :)

0
dimar hanungβ€’ 560

@dimar-hanung

Posted

Hi.. πŸ‘‹, Congratulations on completing the challenge πŸŽ‰ .

I have some interest and feedback with your code

That i like:

  1. I appreciate the similarity of your results with the design, a bit different in scale but still good
  2. html is pretty good, not too nested πŸ‘
  3. Responsive until galaxy fold screen size πŸ‘Œ
  4. CSS Naming is also good, represent what is it for, like <div class="card"> for card section

My Feedback:

  • I suggest you use html semantic convention, for example <div class="container"> to <main class="container">, it will make it clearer, and will improve seo if you want to submit your website to google, i recomended this article: here

  • you can seperate file by folder to be more structured, for example:

public/
β”œβ”€ images/
β”‚  β”œβ”€ qr-code-image.png
β”œβ”€ styles/
β”‚  β”œβ”€ main.css
β”œβ”€ favicon.png
index.html

anyways overall is good, nice solution, hope it useful πŸ™Œ

Marked as helpful

0

@SharonNabiryo

Posted

Thank you so much @dimar-hanung. I appreciate your feedback and I'll make the changes

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