Skip to content
Submitted about 3 years ago

Mobile-first using CSS flexbox

Amandeep715•80
@Amandeep715
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Order summary component
  • HTML
  • CSS
1newbie

Design comparison

SolutionDesign

Solution retrospective

Hello everyone, this is my first ever Front end mentor project. Before this i have only done some freecodecamp projects. So all and any feedback and advice that can help me improve my skills are welcomed(especially on the usage of divs and class names).

Community feedback

  • Account deleted

    You are also missing the curved background shape at the top of the background.

    Marked as helpful
    0
    Amandeep715•80
    @Amandeep715Posted about 3 years ago

    @thulanigamtee Thanks for informing me. I don't how i completely forgot about that part.

    0
  • Kiran Rai•25
    @raikiran07Posted about 3 years ago

    you have two accessibility issues, try to find the errors and fix it. This is the only way to become a good programmer.

    Marked as helpful
    0
    Amandeep715•80
    @Amandeep715Posted about 3 years ago

    @raikiran07 First of all thanks for the feedback. And i did make some changes in the code, but i am not quiet sure if that's what you were referring to.

    0

Please log in to post a comment

Log in with GitHub

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