Nguyen Duc Tan• 240
@tannguyenk3
Submitted
P
David Blackman
@defenstration
All comments
- PDavid Blackman• 60
@defenstration
Posted
It looks pretty good. The only issues I see are with the color in your prep section and some minor spacing issues that make some text look a bit muddled. Pretty minor stuff.
Keep up the good work and have fun!
0 - Journey-Grinder• 150
@Journey-Grinder
Submitted
PDavid Blackman• 60@defenstration
Posted
Your project looks great. I like that you added your own personal touches.
You didn't use the flagposts (h1, main, etc) in your code. When you submitted did it give you an error?
Good work and have fun on your next project?
0 - Mateusz• 390
@MatPawluk
Submitted
PDavid Blackman• 60@defenstration
Posted
Your solution looks great.
I saw that you used a margin utility class. I hadn't thought of that, but it's a great idea for a layout like this.
Is there a reason for using <div class = "main"> instead of <main>?
Keep up the good work and have fun!
0 @MohammadMemon
Submitted
PDavid Blackman• 60@defenstration
Posted
@MohammadMemon
Your project looks good. There is a little less whitespace than the original design, but otherwise good work.
Would having the h1 following another element, the img, cause anything funny to happen with screen readers? I'm new to designing for them, so I'm not sure of the nuances, but I thought a lot of them skipped straight to the h1 when reading the page.
Keep up the good work and have fun!
0
