Skip to content
Submitted about 3 years ago

First mobile approach using semantic css, flexbox and media queries

Mr Mohamed Taha Nassir•10
@MOTANAFI
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

hi, folks I would like to know your opinions and about my first project and also how far it's identical to the origin design and i will appreciate your feedback thank you.

Community feedback

  • CurlyyBraces•55
    @GabiruuPosted about 3 years ago

    Maybe the color of orderbody can be more whitened, to destac the text, out this detail, you does a good job

    Marked as helpful
    0
  • Fluffy Kas•7,715
    @FluffyKasPosted about 3 years ago

    Hey, it's a really good solution! The only thing I'd suggest is that you look into alt attributes! One of them is missing and the other one isn't functional as it is (illustration-hero). You could read this article to have a better understanding: (https://axesslab.com/alt-texts/)

    Marked as helpful
    0
    Mr Mohamed Taha Nassir•10
    @MOTANAFIPosted about 3 years ago

    @FluffyKas I appreciate your feedback!! i included the alt:attribute to both images!

    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