Skip to content
OverviewSolutions
8
Comments
4
Sebastian
@MecosLB

All comments

  • Sebastianā€¢100
    @MecosLBPosted almost 2 years ago

    Hey Stephanie! Well done, your solution looks amazing! āœØ

    āš« Just a reminder to use the :hover pseudo-class to achieve the active state effects some components as the buttons require on this challenges. šŸ˜‰

    āš« Also, don't forget to full customize the default footer provided, I can see you put your name on it but there's no url people could get in touch with. It is useful! šŸ˜œ

    šŸ”µ Extra tip, use PixelPerfect extension so you can have a real comparision with the reference and your work. šŸ™Œ

    Keep coding šŸ’»

    1
  • Sebastianā€¢100
    @MecosLBPosted almost 2 years ago

    Hey Brandon! Nice job, your solution looks pretty cool! šŸ‘

    Just a reminder to use the :hover pseudo-class to achieve the active state effects some components as the buttons require on this challenges. šŸ˜‰

    Also, great job using display: flex; flex-direction: column;, but sometime there's no need to use it. E.g: <button> with display: block; can achieve the same positioning you were looking for with the flex-direction: column; šŸ‘Œ

    Keep coding šŸ’»

    0
  • IbnuKhathirā€¢40
    @IbnuKhathirSubmitted almost 2 years ago

    I feel like I didn't satisfy with my result. I think there is a better way to code this. Can anyone give me a feedback which part I can get better? So I can do more riset and learn at that specific problem. Thank you, Btw, sorry for my bad english

    Learning Order summary card

    2
    Sebastianā€¢100
    @MecosLBPosted almost 2 years ago

    Hey buddy, nice job! :D I recommend you to read carefully the style-guide.md file every time you do a challenge, that file specifies the width the reference's designs are made for. Also, you can find there the fonts (I saw in your html the font's link is invalid, check that out too) and weights you need to use and even the standard font size for that challenge. Extra tip, use PixelPerfect extension so you can have a real comparision with the reference and your work. :D

    Marked as helpful
    0
  • Sebastianā€¢100
    @MecosLBPosted almost 2 years ago

    Nice solution, the only problem i could see when reducing the viewport width is that you're using a margin-left: 40%, this causes your card not to be centered at some point. I'd like to recommend you to use margin auto on left-rigth if you wanna keep centered your card, of course there are other options like justify-content: center if you're using a display flex o something like that. :)

    0