Skip to content
OverviewSolutions
5
Comments
4
Oluwole Kosile-Palmer
@Crea8Boss

All comments

  • Oluwole Kosile-Palmer•90
    @Crea8BossPosted over 1 year ago

    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
  • Oluwole Kosile-Palmer•90
    @Crea8BossPosted almost 2 years ago

    Good job Romeu. Just a few tips for you:

    1. Consider wrapping your classes .card and .attribution in the semantic element <main>.
    2. 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
  • Oluwole Kosile-Palmer•90
    @Crea8BossPosted almost 2 years ago

    Also, looking at the generated screenshot, my solution is not aligned in the center as it should be. What could be the reason?

    0
  • Oluwole Kosile-Palmer•90
    @Crea8BossPosted almost 2 years ago

    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