Skip to content
OverviewSolutions
6
Comments
6
omarsafti09
@omarsafti09

All comments

  • omarsafti09•90
    @omarsafti09

    Great Work!!

  • DolbyTheSheep•120
    @DolbyTheSheep
    What specific areas of your project would you like help with?

    I'd like a review of my use of flexbox.

    • Did I use it correctly?
    • Is there a better way to align all the content?
    • What would you have done different?

    Responsive Four Card Feature

    1
    omarsafti09•90
    @omarsafti09

    Your use of flexbox is good.

    You should structure your style file better.

    Maybe add few comments

  • Terez Lagova•410
    @TerezL
    What are you most proud of, and what would you do differently next time?

    quick challenge, first time doing mobile first

    Product preview

    1
    omarsafti09•90
    @omarsafti09

    Great Work. However, I think the CSS could have been cleaned up

  • Tafadzwa Ruzvidzo•50
    @Taffy-R
    What are you most proud of, and what would you do differently next time?

    My coding ability are improving as I do more projects especially this one

    What challenges did you encounter, and how did you overcome them?

    I didn't encounter any problem in this project I think I'm getting good at this.

    Social links profile using HTML and CSS

    1
    omarsafti09•90
    @omarsafti09

    Good Job!

  • omarsafti09•90
    @omarsafti09

    Good Work.

    I think you could've simplified the CSS a bit.

  • omarsafti09•90
    @omarsafti09

    The implementation is so close to the design. Great work.

    Here are few recommendations:

    1. Use p, h2 tags instead of leaving text in the div tag.
    2. Put both texts in 1 parent div.
    Marked as helpful