Simon Capillon• 100
@simon7195
Submitted
omarsafti09
@omarsafti09
All comments
- omarsafti09• 90
@omarsafti09
Posted
Great Work!!
0 - DolbyTheSheep• 120
@DolbyTheSheep
Submitted
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?
omarsafti09• 90@omarsafti09
Posted
Your use of flexbox is good.
You should structure your style file better.
Maybe add few comments
0 - Terez Lagova• 380
@TerezL
Submitted
What are you most proud of, and what would you do differently next time?
quick challenge, first time doing mobile first
omarsafti09• 90@omarsafti09
Posted
Great Work. However, I think the CSS could have been cleaned up
0 @Taffy-R
Submitted
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.
omarsafti09• 90@omarsafti09
Posted
Good Job!
0- NG• 40
@nguernse
Submitted
omarsafti09• 90@omarsafti09
Posted
Good Work.
I think you could've simplified the CSS a bit.
0 - Arsh• 120
@arsh-de
Submitted
omarsafti09• 90@omarsafti09
Posted
The implementation is so close to the design. Great work.
Here are few recommendations:
- Use p, h2 tags instead of leaving text in the div tag.
- Put both texts in 1 parent div.
Marked as helpful
1
