P
Oluwatoni Odetola• 320
@ToniHunter274
Submitted
Rate me, correct me. I really appreciate it.
Oluwole Kosile-Palmer
@Crea8Boss
All comments
@Crea8Boss
Posted
Good attempt Oluwatoni!
I noted the following adjustments you could make to improve the output:
Accessibility: Add "alt" attribute to your <img> elements. Semantic HTML: Consider wrapping your .container in <main> tag. Styling: Give "body" a background color so that the .container can pop up. Responsiveness: Add media query to make your design responsive on small screens.
Well done!
Marked as helpful
0- Romeu maier• 70
@Edsonromeumaier
Submitted
@Crea8Boss
Posted
Good job Romeu. Just a few tips for you:
- Consider wrapping your classes .card and .attribution in the semantic element <main>.
- You should ensure you include at least a <h1> element in your content in line with best practice. Your .header could be a <h1> content.
Cheers!
Marked as helpful
1 @Crea8Boss
Submitted
@Crea8Boss
Posted
Also, looking at the generated screenshot, my solution is not aligned in the center as it should be. What could be the reason?
0@Crea8Boss
Submitted
@Crea8Boss
Posted
I struggled with applying the background image. I had to use a grey background when I couldn't figure it out. I'm hoping I can obtain hints from this community on how to resolve this.
0
