Skip to content
OverviewSolutions
13
Comments
5
Amirreza
@AmirezaMahmoudi

All comments

  • Amirreza•270
    @AmirezaMahmoudiPosted 8 months ago

    nice

    0
  • Amirreza•270
    @AmirezaMahmoudiPosted 8 months ago

    Awesome

    1
  • Amirreza•270
    @AmirezaMahmoudiPosted 8 months ago

    pixel perfect 👌

    1
  • Ruben•250
    @rugarcia25Submitted 8 months ago
    What are you most proud of, and what would you do differently next time?

    The concepts were very clear

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

    I had a hard time finding the right size for the component background. Testing dimensions until finding the right one.

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

    Would be good get some feedback about the responsive design.

    Responsive QR Code Component

    3
    Amirreza•270
    @AmirezaMahmoudiPosted 8 months ago

    Hello Ruben you missed "." when using css class selector that's why the background color didn't apply. and try to use border-radius for both your img element and main container

    Marked as helpful
    1
  • Aditya•20
    @ipv4adityaSubmitted 9 months ago

    I am not sure if it can be done in a few lines or not, but for me, it took 55 lines of CSS and 23 lines of HTML.

    Positioning the width and height of the card was somewhat challenging, but I anyway completed.

    Please provide feedback if any 🙏

    QR Code Page

    2
    Amirreza•270
    @AmirezaMahmoudiPosted 9 months ago

    nice job my recommendation(opinion) is to use h1 tag instead of p for the first sentence . what i usually do when i want to put contents in the middle of page i set a 100vh height for the body element then use flex to align it. i think you can make some changes that will make it more similar to the actual design like use the background color and text colors provided in the style guide.

    0