Skip to content
OverviewSolutions
6
Comments
6
omarsafti09
@omarsafti09

All comments

  • omarsafti09•90
    @omarsafti09Posted 4 months ago

    Great Work!!

    0
  • DolbyTheSheep•120
    @DolbyTheSheepSubmitted 4 months ago
    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
    @omarsafti09Posted 4 months ago

    Your use of flexbox is good.

    You should structure your style file better.

    Maybe add few comments

    0
  • Terez Lagova•410
    @TerezLSubmitted 4 months ago
    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
    @omarsafti09Posted 4 months ago

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

    0
  • Tafadzwa Ruzvidzo•50
    @Taffy-RSubmitted 4 months ago
    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
    @omarsafti09Posted 4 months ago

    Good Job!

    0
  • omarsafti09•90
    @omarsafti09Posted 4 months ago

    Good Work.

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

    0
  • omarsafti09•90
    @omarsafti09Posted 4 months ago

    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
    1