Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

All comments

  • Kiara• 130

    @kxtara

    Posted

    Hi!

    You've done really well on this challenge, but I have two suggestions.

    • You should add text to your alt attributes for accessibility reasons.
    • The section with the class of section-featured should have a heading tag like h2 - h6 for HTML validation.

    HAPPY CODING

    0
  • Kiara• 130

    @kxtara

    Posted

    Hi!!

    I watched this video, and it really helped me out with the overlay hover effect. If you're still having trouble after watching this, I'm willing to explain further how I accomplished the hover effect.

    HAPPY CODING!!

    0
  • Shaheed Wajee• 220

    @wajeeNPC

    Submitted

    This website was one of the most challenging website that I came through and the mobile view of this was quite hard and I still expect there should be more things to improve on, Please let me know if there's any fault, Thank you

    Kiara• 130

    @kxtara

    Posted

    Hi!!

    For accessibility reasons, I suggest you change the <div> with the class of main to <main> or you can put your code inside of a main tag instead of changing the div.

    HAPPY CODING!!!

    0
  • Kiara• 130

    @kxtara

    Posted

    Looks good! Have you tried changing the first h3 tag to an h1 for accessibility?

    Happy Coding

    0
  • Kiara• 130

    @kxtara

    Posted

    Looks good! Have you tried using h1 instead of h2 to make the text look a little bit more similar...Happy coding!!

    1