Skip to content
OverviewSolutions
3
Comments
3
GeorgeBurn94
@Georgeburn94

All comments

  • garnetcodes•60
    @garnetcodesSubmitted 26 days ago
    What are you most proud of, and what would you do differently next time?

    Having the elements in my finger tip. Growth in knowledge of measurement

    What challenges did you encounter, and how did you overcome them?

    Adding a hover effect to multiple classes. I made research

    What specific areas of your project would you like help with?

    Responsiveness of page

    Used Flexbox

    1
    GeorgeBurn94•50
    @Georgeburn94Posted 25 days ago

    Your project is looking great! The accessibility checks seem solid, and the responsiveness holds up well in dev tools—nice work! The code is well-structured and clean, but adding more comments could make it even easier to follow, especially for future updates. One thing to note is that your version is a bit smaller than the original design. If that’s intentional, no worries, but if you want it to match more closely, consider tweaking spacing and scaling elements up slightly. Overall, awesome job! 🚀

    0
  • P
    Jonatan Samuelsson•490
    @jonatan-samuelssonSubmitted 28 days ago
    What specific areas of your project would you like help with?

    Even though I check all the sizes (cropping the design files to get accurate widths at given screen widths etc, setting fonts to speced pixels, it differs in size, and I don't know why

    Some responsive units and object-fit to crop img

    1
    GeorgeBurn94•50
    @Georgeburn94Posted 28 days ago

    Hey Jonathan,

    Great job on the project so far! You nailed the sizing perfectly, which is awesome to see. I just wanted to share a few friendly notes to help align things a bit closer to the original design and tidy up the code:

    1. Design Tweaks:

      • The underline on the heading isn’t present in the original design, so you can remove that.
      • The font and text size are slightly different from the design—adjusting these will help match the look and feel.

    2. HTML Feedback:

      • You’ve got a small HTML thing to clean up: the role="main" is redundant because the <main> element already tells screen readers it’s the main content. You can simplify it like this: 
        <main>
      • (This is optional, but it’s a good practice to keep your code clean and avoid duplication!)

    You’re doing really well, and these small tweaks will make your project even stronger. :)

    Marked as helpful
    0
  • Elizeu Lima•10
    @elizeu-limaSubmitted 29 days ago
    What are you most proud of, and what would you do differently next time?

    Do que você mais se orgulha e o que faria diferente da próxima vez?

    Na próxima vez quero utilizar algum framework ex: react!

    What challenges did you encounter, and how did you overcome them?

    Quais desafios você encontrou e como os superou?

    no momento não encontrei ainda um grande desafio, mas foi muito prazeroso e de grande aprendizado fazer este projeto.

    What specific areas of your project would you like help with?

    Com quais áreas específicas do seu projeto você gostaria de ajuda?

    Eu tenho dificuldade na parte criativa, gostaria de ideias de como evoluir e fazer coisas mais interessantes, eu consigo ser detalhista e replicar o projeto mas tenho dificuldade com a parte criativa.

    Desafio Qr Code Frontend Mentor

    #pure-css#jss
    1
    GeorgeBurn94•50
    @Georgeburn94Posted 28 days ago

    You got the sizing perfectly right!

    The padding is slightly too big, but this could be easily fixed.

    Accessibility looks good on all sizes :)

    0