Skip to content
Submitted about 2 years ago

Order summary component

jom•130
@Jmlsoteyza
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

I'm still having a problem with display flex but because of that, I've started to learn more about using the flex. I do mistakes here, and I can see them. I'll probably to change them in my next solutions. Feel free to point out my mistakes, I'll gladly to know it!!

Community feedback

  • Abdullah•340
    @ASH2001princePosted about 2 years ago

    Hello jom congratulations for your new project -I have some HTML tips for you:

    1. you can use the <section> or <article> elements to gather the elements like the <h1>,<p> and <img> together.
    2. instead of coping the <svg> you can import the image by using <img> element.
    3. <footer> usages for the last informations like names some links maybe and we can put the <address> attribute inside it as well.
    Marked as helpful
    1
    jom•130
    @JmlsoteyzaPosted about 2 years ago

    @ASH2001prince Thankyou bro, but do you have any idea why my "Annual plan" and the other texts changed for some reason? like it gets big compare to my site.

    0
    Abdullah•340
    @ASH2001princePosted about 2 years ago

    @Jmlsoteyza yes I think I do if you would put the music icon and the "Annual Plan" and "change" link in one div or section without the others and remove all the extra heights it well be smaller and for a last thing you can align the items using the flex system. hope you find it helpful

    Marked as helpful
    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