@Abdelrahman1Ibrahim
Submitted
Louai Khodary
@LouaiKhodary13
All comments
- Louai Khodary• 350
@LouaiKhodary13
Posted
Great job on finishing the challenge, but I think you should have more width to the main container to be pixel perfect
0 - Erick Felipa Pineda• 160
@Efelipa
Submitted
What are you most proud of, and what would you do differently next time?
How to use grid-area properly.
What challenges did you encounter, and how did you overcome them?
Create the card sample, but once it was done, it was just a matter of changing the information.
Louai Khodary• 350@LouaiKhodary13
Posted
Congrats on completing the challenge But I think You forgot to add border radius to the cards
0 - SaidBC• 150
@SaidBC
Submitted
Louai Khodary• 350@LouaiKhodary13
Posted
Congrats on finishing the challenge
1 - Oswaldo Castro• 240
@tufcoder
Submitted
What are you most proud of, and what would you do differently next time?
I made a simple html not over using divs.
What challenges did you encounter, and how did you overcome them?
All the work is done from eye, I just happy to finish closest to the example.
What specific areas of your project would you like help with?
I think that I need help in Responsive images.
Louai Khodary• 350@LouaiKhodary13
Posted
Congrats on finishing the project, but I think you gave the parent container a larger width than the design
0 - Maximilian Dybvik• 200
@RegexRiddler
Submitted
What are you most proud of, and what would you do differently next time?
Getting pretty close to the design, and styling the list markers. Next time I would restructure the styling using SASS and stick to BEM naming convention.
What challenges did you encounter, and how did you overcome them?
Styling the list markers and table. I still haven't overcome them. They aren't perfect and I'm wondering if it even is possible without removing the markers and make new ones as a span element.
What specific areas of your project would you like help with?
Any feedback is appreciated.
- Is my code readable and maintainable?
- Is my use of the section element correct for this application?
- Anything that sticks out that i should fix or improve?
Louai Khodary• 350@LouaiKhodary13
Posted
Congrats on finishing the challenge
0 - Antwaun Johnson• 40
@antwaunjohnson
Submitted
What specific areas of your project would you like help with?
I open to any constructive feedback.
Louai Khodary• 350@LouaiKhodary13
Posted
Great Job on finishing the challenge, but I think the profile image should have less width and height to match the design
Marked as helpful
1 - Leonardo Gusmão• 10
@gitGusmao
Submitted
Louai Khodary• 350@LouaiKhodary13
Posted
Great job on finishing the challenge, but I think you need to add a margin bottom of 15 px to the image
0 - Ryan• 130
@ryanxora
Submitted
Louai Khodary• 350@LouaiKhodary13
Posted
Congrats on finishing the project, but I think the Learning after the Image needs to have a little more padding
Marked as helpful
0 What are you most proud of, and what would you do differently next time?
My first project from Frontendmentor.io, so yeah.. this is my little step towards a pro-full-stack dev (I hope) 🔥
Louai Khodary• 350@LouaiKhodary13
Posted
congrats keep up the good work
0- Harpreet Singh• 50
@hrps404
Submitted
What are you most proud of, and what would you do differently next time?
I used few new animations this time and box shadows were also something new for me. Althouth I can't say proud because there's a lot more to learn.
Louai Khodary• 350@LouaiKhodary13
Posted
Congrats on finishing the challenge, but I think you need to add a little bit of box shadow try this box-shadow: 0.5rem 0.5rem black;
0
