Skip to content
OverviewSolutions
4
Comments
20
Luke
@lukebergmann

All comments

  • Luke•220
    @lukebergmannPosted almost 4 years ago

    Hey Abbas,

    This code is looking really great! The others to reply definitely covered a ton! The one thing I would double check it the font-color of the 'register' button. I believe you chose the pink color when it should be the plum color! Other than that, this is looking dope!

    Happy Coding! Luke

    1
  • Luke•220
    @lukebergmannPosted almost 4 years ago

    Hey there Mohamed,

    This code is looking fantastic. As mentioned above, the font-size at the top heading could have a little bit of work. The box-shadow is could be a little more noticeable as well!

    Happy coding!

    Luke

    1
  • Luke•220
    @lukebergmannPosted almost 4 years ago

    Hey @AuiPaul,

    This code is looking really nice! I like the fact that you added your own spin to it. It's the best way to learn haha. I would suggest tweaking the size of the profile card. It is looking slightly more rectangular on the vertical axis. This gives more of a stretched illusion that seems claustrophobic. Making it more square adds a bit of breathing room. All in all, this is a pretty good code!

    Happy coding my friend, Luke

    0
  • Luke•220
    @lukebergmannPosted almost 4 years ago

    Hey Tyler!

    Your code is looking pretty spot on! I can't think of anything to add other than, keep it up. You're killing it!

    Happy coding, Luke

    1
  • Jhonatan•40
    @JhonnK08Submitted almost 4 years ago

    Hi for all!

    I did it, but was a little difficult because I didn't know much about flexbox.

    I would like some feedbacks, improvements and what else you want.

    Thanks! :)

    HTML, CSS and Flexbox challenge

    2
    Luke•220
    @lukebergmannPosted almost 4 years ago

    Hey Johnathan!

    This was the first challenge that I completed and I definitely had difficulties as well haha. None the less, this looks great! I would suggest making minor tweaks to the profile card shape. Right now, it is looking slightly long vertically. This gives it a stretched look with not a lot of breathing room within the card. I would tuck in the 'London' word closer to Victor Crest to give the bottom portion a little more space. You also may want to consider throwing a box-shadow on that bad boy and you're set!

    Happy Coding, Luke

    1
  • Luke•220
    @lukebergmannPosted almost 4 years ago

    Yo, this looks pretty much identical!

    Happy coding! Luke

    1
  • Luke•220
    @lukebergmannPosted almost 4 years ago

    Hey Nicole,

    Your solution is looking fantastic! Honestly, the only thing that stands out to me is the height and boldness of your 'horizontal rule' above the logistics section. It has an effect that almost looks stuffy and high up. Lowering the hr slightly and making it thinner will give the entire profile card more breathing room!

    Happy Coding,

    Luke

    0
  • Luke•220
    @lukebergmannPosted almost 4 years ago

    Hey Ryan!

    This is looking pretty spot on. The sizing is looking great. Two things that I would suggest would be first, to size the main box a little more rectangular. It is seeming a little squishy the more square the shape. A rectangular shape may give the whole layout a little bit of room to breath. Secondly, I would focus my attention onto the harshness of the box shadow. In the Design version, the shadow drops from the top whereas yours sort of encompasses the entire box. Eliminating the top shadow will have more of an believable and crisp effect!

    Honestly, it's looking pretty dope, keep it up!

    Happy coding, Luke

    0
  • Luke•220
    @lukebergmannPosted almost 4 years ago

    Hey Akshay!

    This looks really good! I would suggest working a little bit on the horizontal rule. It is seeming a little dark compared to the original. Also, a shadow behind the card would definitely make it pop off the screen a bit more!

    0
  • Luke•220
    @lukebergmannPosted almost 4 years ago

    This is great! I can't decide if I like yours or the original better!

    0
  • Luke•220
    @lukebergmannPosted almost 4 years ago

    This looks fantastic. I would consider working a little bit on the spacing between the subheading (beneath "10000+ text) as well as a little less spacing between the purple review boxes! Other than that, this looks almost identical!

    0