Skip to content
OverviewSolutions
14
Comments
807

Anosha Ahmed

@anoshaahmedSelf-taught Web Developer9,300 points

Let's​​​​​　connect​​​​​　on​​​​​　Linkedin​​​​​　&　GitHub!

Latest solutions

View more solutions

Latest comments

  • Anosha Ahmed•9,300
    @anoshaahmed

    5-6hrs is quick. good job!

  • Anosha Ahmed•9,300
    @anoshaahmed

    Aim to use semantic tags when writing html. I would change div class="card" to main class="card". I'll also wrap the image in an h1. I see that you wrote two separate codes for the two screen sizes. Instead of that, I would recommend that you write the mobile code as your base code, then make a media query for the bigger size. For example, you can give a max-width to something, and then in the media query for the bigger size, you would just increase the max-width. That way you're building a responsive design but also doing less work

    you did great though. that's ^ just a way for u to upgrade ur behind-the-scene skills

  • Anosha Ahmed•9,300
    @anoshaahmed

    hey good job on this challenge! your report above shows a few issues though:

    • you should have at least one <h1> in your webpage

    • add alt="" to your <img> tags

    here is a list i made of accessibility issues & best practices

    hope this helps :))

    Marked as helpful
  • Anosha Ahmed•9,300
    @anoshaahmed

    hey good job on this challenge! you should have at least one <h1> in your webpage

    here is a list i made of accessibility issues & best practices

    hope this helps :))

    Marked as helpful
  • Anosha Ahmed•9,300
    @anoshaahmed

    hey good job on this challenge! your report above shows a few issues though:

    • wrap everything in your body in <main> OR use semantic tags! if you're using div right after body, then give them a role=""; however, it's a little frowned upon to use role right after body, so try to stick with semantic tags instead. you can read more about landmarks here

    • add alt="" to your <img> tags

    here is a list i made of accessibility issues & best practices

    hope this helps :))

    Marked as helpful
  • Anosha Ahmed•9,300
    @anoshaahmed

    hey good job on this challenge! wrap everything in your body in <main> OR use semantic tags! if you're using div right after body, then give them a role=""; however, it's a little frowned upon to use role right after body, so try to stick with semantic tags instead. you can read more about landmarks here

    here is a list i made of accessibility issues & best practices

    hope this helps :))

    Marked as helpful
View more comments